Merge branch 'symlink_hooks_directory' into 'master'

Symlink hooks directory

See merge request !44

2 files changed, 7 insertions, 4 deletions

diff --git a/CHANGELOG b/CHANGELOG
index daf6d11..30bddec 100644
--- a/CHANGELOG
+++ b/CHANGELOG
@@ -3,6 +3,7 @@ v2.0.0
   - Replace raise with abort when checking path to prevent path exposure
   - Handle invalid number of arguments on remote commands
   - Replace update hook with pre-receive and post-receive hooks. 
+  - Symlink the whole hooks directory
   - Ignore missing repositories in create-hooks
   - Connect to Redis via sockets by default
 
diff --git a/lib/gitlab_projects.rb b/lib/gitlab_projects.rb
index d6922c6..58bbd28 100644
--- a/lib/gitlab_projects.rb
+++ b/lib/gitlab_projects.rb
@@ -5,6 +5,8 @@ require_relative 'gitlab_config'
 require_relative 'gitlab_logger'
 
 class GitlabProjects
+  GLOBAL_HOOKS_DIRECTORY = File.join(ROOT_PATH, 'hooks')
+
   # Project name is a directory name for repository with .git at the end
   # It may be namespaced or not. Like repo.git or gitlab/repo.git
   attr_reader :project_name
@@ -18,10 +20,10 @@ class GitlabProjects
   attr_reader :full_path
 
   def self.create_hooks(path)
-    %w(pre-receive post-receive).each do |hook_name|
-      hook = File.join(path, 'hooks', hook_name)
-      File.delete(hook) if File.exists?(hook)
-      File.symlink(File.join(ROOT_PATH, 'hooks', hook_name), hook)
+    local_hooks_directory = File.join(path, 'hooks')
+    unless File.realpath(local_hooks_directory) == File.realpath(GLOBAL_HOOKS_DIRECTORY)
+      FileUtils.mv(local_hooks_directory, "#{local_hooks_directory}.old.#{Time.now.to_i}")
+      FileUtils.ln_s(GLOBAL_HOOKS_DIRECTORY, local_hooks_directory)
     end
   end