Merge pull request #186 from cirosantilli/restore-path

Restore hooks PATH before calling ruby.

CHANGELOG

+v2.3.0
+  - Restore PATH before calling hooks
+
 v2.2.0
   - Support for custom hooks (Drew Blessing and Jose Kahan)
 

hooks/post-receive

-#!/usr/bin/env ruby
+#!/usr/bin/env bash
 
-# This file was placed here by GitLab. It makes sure that your pushed commits
-# will be processed properly.
-
-refs = ARGF.read
-key_id  = ENV['GL_ID']
-repo_path = Dir.pwd
-
-require_relative '../lib/gitlab_custom_hook'
-require_relative '../lib/gitlab_post_receive'
-
-if GitlabPostReceive.new(repo_path, key_id, refs).exec &&
-    GitlabCustomHook.new.post_receive(refs, repo_path)
-  exit 0
-else
-  exit 1
-end
+# Remove paths Git automatically prepends to PATH on hooks.
+export PATH="$GITLAB_PATH_OUTSIDE_HOOK"
+ruby "$GIT_DIR/hooks/post-receive-main.rb"

hooks/post-receive-main.rb

+#!/usr/bin/env ruby
+
+# This file was placed here by GitLab. It makes sure that your pushed commits
+# will be processed properly.
+
+refs = ARGF.read
+key_id  = ENV['GL_ID']
+repo_path = Dir.pwd
+
+require_relative '../lib/gitlab_custom_hook'
+require_relative '../lib/gitlab_post_receive'
+
+if GitlabPostReceive.new(repo_path, key_id, refs).exec &&
+    GitlabCustomHook.new.post_receive(refs, repo_path)
+  exit 0
+else
+  exit 1
+end

hooks/pre-receive

-#!/usr/bin/env ruby
+#!/usr/bin/env bash
 
-# This file was placed here by GitLab. It makes sure that your pushed commits
-# will be processed properly.
-
-refs = ARGF.read
-key_id  = ENV['GL_ID']
-repo_path = Dir.pwd
-
-require_relative '../lib/gitlab_custom_hook'
-require_relative '../lib/gitlab_access'
-
-if GitlabAccess.new(repo_path, key_id, refs).exec &&
-    GitlabCustomHook.new.pre_receive(refs, repo_path)
-  exit 0
-else
-  exit 1
-end
+# Remove paths Git automatically prepends to PATH on hooks.
+export PATH="$GITLAB_PATH_OUTSIDE_HOOK"
+ruby "$GIT_DIR/hooks/pre-receive-main.rb"

hooks/pre-receive-main.rb

+#!/usr/bin/env ruby
+
+# This file was placed here by GitLab. It makes sure that your pushed commits
+# will be processed properly.
+
+refs = ARGF.read
+key_id  = ENV['GL_ID']
+repo_path = Dir.pwd
+
+require_relative '../lib/gitlab_custom_hook'
+require_relative '../lib/gitlab_access'
+
+if GitlabAccess.new(repo_path, key_id, refs).exec &&
+    GitlabCustomHook.new.pre_receive(refs, repo_path)
+  exit 0
+else
+  exit 1
+end

hooks/update

-#!/usr/bin/env ruby
+#!/usr/bin/env bash
 
-# This file was placed here by GitLab. It makes sure that your pushed commits
-# will be processed properly.
-
-ref_name = ARGV[0]
-old_value = ARGV[1]
-new_value = ARGV[2]
-repo_path = Dir.pwd
-
-require_relative '../lib/gitlab_custom_hook'
-
-if GitlabCustomHook.new.update(ref_name, old_value, new_value, repo_path)
-  exit 0
-else
-  exit 1
-end
+# Remove paths Git automatically prepends to PATH on hooks.
+export PATH="$GITLAB_PATH_OUTSIDE_HOOK"
+ruby "$GIT_DIR/hooks/update-main.rb"

hooks/update-main.rb

+#!/usr/bin/env ruby
+
+# This file was placed here by GitLab. It makes sure that your pushed commits
+# will be processed properly.
+
+ref_name = ARGV[0]
+old_value = ARGV[1]
+new_value = ARGV[2]
+repo_path = Dir.pwd
+
+require_relative '../lib/gitlab_custom_hook'
+
+if GitlabCustomHook.new.update(ref_name, old_value, new_value, repo_path)
+  exit 0
+else
+  exit 1
+end