Merge branch 'remove_empty_post_receive' of /home/git/repositories/gitlab/gitlab-shell

hooks/post-receive

-#!/usr/bin/env ruby
-
-# This file was placed here by GitLab.
-# You can put your own hook here, but be careful when updating gitlab-shell!

lib/gitlab_projects.rb

@@ -182,9 +182,6 @@ class GitlabProjects
   private
 
   def create_hooks_to(dest_path)
-    pr_hook_path = File.join(ROOT_PATH, 'hooks', 'post-receive')
-    up_hook_path = File.join(ROOT_PATH, 'hooks', 'update')
-
-    "ln -s #{pr_hook_path} #{dest_path}/hooks/post-receive && ln -s #{up_hook_path} #{dest_path}/hooks/update"
+    "ln -s #{File.join(ROOT_PATH, 'hooks', 'update')} #{dest_path}/hooks/update"
   end
 end

spec/gitlab_projects_spec.rb

@@ -101,7 +101,6 @@ describe GitlabProjects do
 
     it "should receive valid cmd" do
       valid_cmd = "cd #{tmp_repo_path} && git init --bare"
-      valid_cmd << " && ln -s #{ROOT_PATH}/hooks/post-receive #{tmp_repo_path}/hooks/post-receive"
       valid_cmd << " && ln -s #{ROOT_PATH}/hooks/update #{tmp_repo_path}/hooks/update"
       gl_projects.should_receive(:system).with(valid_cmd)
       gl_projects.exec
@@ -238,7 +237,6 @@ describe GitlabProjects do
       gl_projects_fork.exec.should be_true
       File.exists?(dest_repo).should be_true
       File.exists?(File.join(dest_repo, '/hooks/update')).should be_true
-      File.exists?(File.join(dest_repo, '/hooks/post-receive')).should be_true
     end
 
     it "should not fork if a project of the same name already exists" do

support/rewrite-hooks.sh

@@ -6,35 +6,21 @@
 home_dir="/home/git"
 src=${1:-"$home_dir/repositories"}
 
+function create_link_in {
+  ln -s -f "$home_dir/gitlab-shell/hooks/update" "$1/hooks/update"
+}
+
 for dir in `ls "$src/"`
 do
   if [ -d "$src/$dir" ]; then
-
-    if [ "$dir" = "gitolite-admin.git" ]
-    then
-      continue 
-    fi
-
     if [[ "$dir" =~ ^.*\.git$ ]]
     then
-      project_hook="$src/$dir/hooks/post-receive"
-      gitolite_hook="$home_dir/gitlab-shell/hooks/post-receive"
-      ln -s -f $gitolite_hook $project_hook
-
-      project_hook="$src/$dir/hooks/update"
-      gitolite_hook="$home_dir/gitlab-shell/hooks/update"
-      ln -s -f $gitolite_hook $project_hook
+      create_link_in "$src/$dir"
     else
       for subdir in `ls "$src/$dir/"`
       do
         if [ -d "$src/$dir/$subdir" ] && [[ "$subdir" =~ ^.*\.git$ ]]; then
-          project_hook="$src/$dir/$subdir/hooks/post-receive"
-          gitolite_hook="$home_dir/gitlab-shell/hooks/post-receive"
-          ln -s -f $gitolite_hook $project_hook
-
-          project_hook="$src/$dir/$subdir/hooks/update"
-          gitolite_hook="$home_dir/gitlab-shell/hooks/update"
-          ln -s -f $gitolite_hook $project_hook
+          create_link_in "$src/$dir/$subdir"
         fi
       done
     fi