On Wed, Dec 17, 2008 at 08:44:40AM +0100, Johannes Sixt <j.sixt@xxxxxxxxxxxxx> wrote: > > This is true, but having the executable bit is necessary as well. I > > think it would be better to just append this requirement instead of > > replacing the old one with this. > > Markus's proposed new wording is correct because the .sample hooks *are* > already executable. I thought about the following situation: The user reads the documentation while working in an older repo (initialized a few versions ago). S/he sees that the .sample suffix is already missing, so s/he assumes that the hook is already active. Which is not true, because the +x bit is missing.
Attachment:
pgpIXaQ0EWhgT.pgp
Description: PGP signature