[PATCH] contrib/hooks: avoid requiring root access in usage instructions
From: Jonathan Nieder <hidden>
Date: 2016-06-15 22:55:04
Subsystem:
the rest · Maintainer:
Linus Torvalds
Comments in hooks/post-receive-email suggest: For example, on debian the hook is stored in /usr/share/git-core/contrib/hooks/post-receive-email: chmod a+x post-receive-email cd /path/to/your/repository.git ln -sf /usr/share/git-core/contrib/hooks/post-receive-email hooks/post-receive Doing that means changing permissions on a file provided by a package, which is problematic in a number of ways: the permissions would be likely to change back in later upgrades, and changing them requires root access. Copying the script into each repo that uses it is not much better, since each copy would be maintained separately and not benefit from bugfixes in the master copy. Better to ship the hook with executable permission and remove the chmod line so enabling the hook becomes a one-step process: just symlink it into place. Likewise for the pre-auto-gc-battery hook. Reported-by: Olivier Berger <redacted> Signed-off-by: Jonathan Nieder <redacted> --- From <http://bugs.debian.org/687391>. Thoughts? Jonathan contrib/hooks/post-receive-email | 1 - contrib/hooks/pre-auto-gc-battery | 1 - 2 files changed, 2 deletions(-) mode change 100644 => 100755 contrib/hooks/pre-auto-gc-battery
diff --git a/contrib/hooks/post-receive-email b/contrib/hooks/post-receive-email
index 8ca6607a..359f1ad2 100755
--- a/contrib/hooks/post-receive-email
+++ b/contrib/hooks/post-receive-email@@ -13,7 +13,6 @@ # For example, on debian the hook is stored in # /usr/share/git-core/contrib/hooks/post-receive-email: # -# chmod a+x post-receive-email # cd /path/to/your/repository.git # ln -sf /usr/share/git-core/contrib/hooks/post-receive-email hooks/post-receive #
diff --git a/contrib/hooks/pre-auto-gc-battery b/contrib/hooks/pre-auto-gc-battery
old mode 100644
new mode 100755
index 1f914c94..9d0c2d19
--- a/contrib/hooks/pre-auto-gc-battery
+++ b/contrib/hooks/pre-auto-gc-battery@@ -13,7 +13,6 @@ # For example, if the hook is stored in # /usr/share/git-core/contrib/hooks/pre-auto-gc-battery: # -# chmod a+x pre-auto-gc-battery # cd /path/to/your/repository.git # ln -sf /usr/share/git-core/contrib/hooks/pre-auto-gc-battery \ # hooks/pre-auto-gc
--
1.8.0.rc2