{"thread":{"id":"28444","subject":"[PATCH] contrib/hooks: adapt default Debian install location for contrib hooks","startedAt":"2011-09-20T11:19:14Z","lastAt":"2011-09-26T12:58:35Z","messageCount":4,"participants":["Gerrit Pape","Drew Northup","Junio C Hamano"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"175857","messageId":"20110920111914.17913.qmail@d29ca435be5a3d.315fe32.mid.smarden.org","threadId":"28444","inReplyTo":null,"subject":"[PATCH] contrib/hooks: adapt default Debian install location for contrib hooks","fromName":"Gerrit Pape","fromEmail":"pape@smarden.org","sentAt":"2011-09-20T11:19:14Z","receivedAt":"2011-09-20T11:19:14Z","isPatch":true,"sender":{"key":"pape@smarden.org","avatar":"https://avatars.githubusercontent.com/u/143170252?v=4"},"body":"Placing the contrib hooks into /usr/share/doc/ wasn't a good idea in the\nfirst place. According to the Debian policy they should be located in\n/usr/share/git-core/, so let's put them there.\n\nThanks to Bill Allombert for reporting this through\n http://bugs.debian.org/640949\n\nSigned-off-by: Gerrit Pape \n---\n contrib/hooks/post-receive-email | 4 ++--\n templates/hooks--post-receive.sample | 2 +-\n 2 files changed, 3 insertions(+), 3 deletions(-)\n\ndiff --git a/contrib/hooks/post-receive-email b/contrib/hooks/post-receive-email\nindex fa6d41a..ba077c1 100755\n--- a/contrib/hooks/post-receive-email\n+++ b/contrib/hooks/post-receive-email\n@@ -11,11 +11,11 @@\n # will have put this somewhere standard. You should make this script\n # executable then link to it in the repository you would like to use it in.\n # For example, on debian the hook is stored in\n-# /usr/share/doc/git-core/contrib/hooks/post-receive-email:\n+# /usr/share/git-core/contrib/hooks/post-receive-email:\n #\n # chmod a+x post-receive-email\n # cd /path/to/your/repository.git\n-# ln -sf /usr/share/doc/git-core/contrib/hooks/post-receive-email hooks/post-receive\n+# ln -sf /usr/share/git-core/contrib/hooks/post-receive-email hooks/post-receive\n #\n # This hook script assumes it is enabled on the central repository of a\n # project, with all users pushing only to it and not between each other. It\ndiff --git a/templates/hooks--post-receive.sample b/templates/hooks--post-receive.sample\nindex 7a83e17..e48346e 100755\n--- a/templates/hooks--post-receive.sample\n+++ b/templates/hooks--post-receive.sample\n@@ -12,4 +12,4 @@\n # see contrib/hooks/ for a sample, or uncomment the next line and\n # rename the file to \"post-receive\".\n \n-#. /usr/share/doc/git-core/contrib/hooks/post-receive-email\n+#. /usr/share/git-core/contrib/hooks/post-receive-email\n-- \n1.7.5.4\n"},{"id":"175884","messageId":"1316547492.14618.10.camel@ddn-tmpdesk.its.maine.edu","threadId":"28444","inReplyTo":"20110920111914.17913.qmail@d29ca435be5a3d.315fe32.mid.smarden.org","subject":"Re: [PATCH] contrib/hooks: adapt default Debian install location for contrib hooks","fromName":"Drew Northup","fromEmail":"drew.northup@maine.edu","sentAt":"2011-09-20T19:38:12Z","receivedAt":"2011-09-20T19:38:12Z","isPatch":true,"sender":{"key":"drew.northup@maine.edu","avatar":"https://avatars.githubusercontent.com/u/18331571?v=4"},"body":"\nOn Tue, 2011-09-20 at 11:19 +0000, Gerrit Pape wrote:\n> Placing the contrib hooks into /usr/share/doc/ wasn't a good idea in the\n> first place. According to the Debian policy they should be located in\n> /usr/share/git-core/, so let's put them there.\n> \n> Thanks to Bill Allombert for reporting this through\n> http://bugs.debian.org/640949\n> \n> Signed-off-by: Gerrit Pape \n> ---\n> contrib/hooks/post-receive-email | 4 ++--\n> templates/hooks--post-receive.sample | 2 +-\n> 2 files changed, 3 insertions(+), 3 deletions(-)\n\nWhat is the rationale for applying this outside of the land of Debian? I\nunderstand the EPEL/Fedora crew are putting those example hooks\ninto /usr/share/git-gore, but I want to know why that should become the\nstandard in git.git.\n\n-- \n-Drew Northup\n________________________________________________\n\"As opposed to vegetable or mineral error?\"\n-John Pescatore, SANS NewsBites Vol. 12 Num. 59\n"},{"id":"175885","messageId":"7vr53b578t.fsf@alter.siamese.dyndns.org","threadId":"28444","inReplyTo":"20110920111914.17913.qmail@d29ca435be5a3d.315fe32.mid.smarden.org","subject":"Re: [PATCH] contrib/hooks: adapt default Debian install location for contrib hooks","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2011-09-20T19:44:18Z","receivedAt":"2011-09-20T19:44:18Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Gerrit Pape writes:\n\n> diff --git a/contrib/hooks/post-receive-email b/contrib/hooks/post-receive-email\n> index fa6d41a..ba077c1 100755\n> --- a/contrib/hooks/post-receive-email\n> +++ b/contrib/hooks/post-receive-email\n> @@ -11,11 +11,11 @@\n> # will have put this somewhere standard. You should make this script\n> # executable then link to it in the repository you would like to use it in.\n> # For example, on debian the hook is stored in\n> -# /usr/share/doc/git-core/contrib/hooks/post-receive-email:\n> +# /usr/share/git-core/contrib/hooks/post-receive-email:\n> #\n> # chmod a+x post-receive-email\n> # cd /path/to/your/repository.git\n> -# ln -sf /usr/share/doc/git-core/contrib/hooks/post-receive-email hooks/post-receive\n> +# ln -sf /usr/share/git-core/contrib/hooks/post-receive-email hooks/post-receive\n\nThis one I do not have any problem with; the leading text makes it very\nclear that this is an example that is specific to Debian and that is \ngood enough to signal readers that they may have to look elsewhere if they\nare on a different distribution.\n\n> diff --git a/templates/hooks--post-receive.sample b/templates/hooks--post-receive.sample\n> index 7a83e17..e48346e 100755\n> --- a/templates/hooks--post-receive.sample\n> +++ b/templates/hooks--post-receive.sample\n> @@ -12,4 +12,4 @@\n> # see contrib/hooks/ for a sample, or uncomment the next line and\n> # rename the file to \"post-receive\".\n> \n> -#. /usr/share/doc/git-core/contrib/hooks/post-receive-email\n> +#. /usr/share/git-core/contrib/hooks/post-receive-email\n\nI am however not sure about this. For one thing, it is not clear that this\n(both old and new) text are meant to be Debian specific, and they should\nnot complain and instead should look for it elsewhere, if the users do not\nfind the sample in the path that is shown in the file. For another, it is\nnot \"see ... for a sample, OR UNCOMMENT\" at all, for readers who are _NOT_\ninterested in sending e-mails from the hook at all, but the way it is\nstated makes it sound as if people who do not want to bother reading the\nsamples can blindly enable the hook and it is guaranteed that the\nparticular sample hook satisfies their needs, which is not the case.\n\nPerhaps removing this sample file might be a better option. The first ten\nlines or so repeat what the documentation says, which can go stale, and if\nwe remove them, the only thing that remains is the \"see contrib/hooks for\na sample\".\n\nThanks.\n"},{"id":"176202","messageId":"20110926125835.26573.qmail@06368f96dac286.315fe32.mid.smarden.org","threadId":"28444","inReplyTo":"7vr53b578t.fsf@alter.siamese.dyndns.org","subject":"[PATCH 1/2] contrib/hooks: adapt comment about Debian install location for contrib hooks","fromName":"Gerrit Pape","fromEmail":"pape@smarden.org","sentAt":"2011-09-26T12:58:35Z","receivedAt":"2011-09-26T12:58:35Z","isPatch":true,"sender":{"key":"pape@smarden.org","avatar":"https://avatars.githubusercontent.com/u/143170252?v=4"},"body":"Placing the contrib hooks into /usr/share/doc/ wasn't a good idea in the\nfirst place. According to the Debian policy they should be located in\n/usr/share/git-core/, so let's put them there.\n\nThanks to Bill Allombert for reporting this through\n http://bugs.debian.org/640949\n\nSigned-off-by: Gerrit Pape \n---\n\nOn Tue, Sep 20, 2011 at 12:44:18PM -0700, Junio C Hamano wrote:\n> > --- a/contrib/hooks/post-receive-email\n> > +++ b/contrib/hooks/post-receive-email\n> > @@ -11,11 +11,11 @@\n> > # will have put this somewhere standard. You should make this\n> > script\n> > # executable then link to it in the repository you would like to\n> > use it in.\n> > # For example, on debian the hook is stored in\n> > -# /usr/share/doc/git-core/contrib/hooks/post-receive-email:\n> > +# /usr/share/git-core/contrib/hooks/post-receive-email:\n> > #\n> > # chmod a+x post-receive-email\n> > # cd /path/to/your/repository.git\n> > -# ln -sf /usr/share/doc/git-core/contrib/hooks/post-receive-email\n> > hooks/post-receive\n> > +# ln -sf /usr/share/git-core/contrib/hooks/post-receive-email\n> > hooks/post-receive\n>\n> This one I do not have any problem with; the leading text makes it\n> very\n> clear that this is an example that is specific to Debian and that is\n> good enough to signal readers that they may have to look elsewhere if\n> they\n> are on a different distribution.\n>\n\nOk, thanks. I stripped this patch to just touch this file.\n\nRegards, Gerrit.\n\n contrib/hooks/post-receive-email | 4 ++--\n 1 files changed, 2 insertions(+), 2 deletions(-)\n\ndiff --git a/contrib/hooks/post-receive-email b/contrib/hooks/post-receive-email\nindex fa6d41a..ba077c1 100755\n--- a/contrib/hooks/post-receive-email\n+++ b/contrib/hooks/post-receive-email\n@@ -11,11 +11,11 @@\n # will have put this somewhere standard. You should make this script\n # executable then link to it in the repository you would like to use it in.\n # For example, on debian the hook is stored in\n-# /usr/share/doc/git-core/contrib/hooks/post-receive-email:\n+# /usr/share/git-core/contrib/hooks/post-receive-email:\n #\n # chmod a+x post-receive-email\n # cd /path/to/your/repository.git\n-# ln -sf /usr/share/doc/git-core/contrib/hooks/post-receive-email hooks/post-receive\n+# ln -sf /usr/share/git-core/contrib/hooks/post-receive-email hooks/post-receive\n #\n # This hook script assumes it is enabled on the central repository of a\n # project, with all users pushing only to it and not between each other. It\n-- \n1.7.6.3\n"}]}