Adding custom hooks to a bare repository.

Previous thread: [PATCH] Allow installing in the traditional way by Junio C Hamano on Monday, July 28, 2008 - 12:09 am. (2 messages)

Next thread: [PATCH] Updated the Documentation/config.txt core.autocrlf configuration setting documentation by Jurko Gospodnetić on Monday, July 28, 2008 - 4:48 am. (1 message)
From: Thomas Adam
Date: Monday, July 28, 2008 - 4:04 am

Hello all,

I'm trying to add in "post-merge" as a valid hook file to an already
created --bare repository.  In adding it to the correct place and
setting the correct permissions on it (the repository is a shared
one), it doesn't seem to be pulled down with the other hook files
already present when doing a "git clone".

Any ideas why?

Indeed, I also, as a test, added this post-merge hook to a repository
and created a --bare repository from that - thinking that perhaps the
hook file had to present at the time of creating a new repository, but
even then it wasn't copied over to the bare repository.

What am I doing wrong?

Thanks in advance,

-- Thomas Adam
--

From: Johannes Schindelin
Date: Monday, July 28, 2008 - 4:26 am

Hi,


Yes.  Hooks, just as the config and reflogs, are supposed to be local 
things.  Rationale being: it is rude, and also insecure, to install 
something that potentially calls other programs without the user saying 
so.

All you can do is checking in a copy of the hook, and ask your users/check 
in your build system that it is installed.

Hth,
Dscho


--

From: Thomas Adam
Date: Monday, July 28, 2008 - 5:32 am

Hmm -- perhaps I am being unintentionally dense, but I am assuming
that when you say "checking in a copy" you mean anywhere other than
.git/hooks/ since that isn't tracked by git.  I have no problem with
the rationale you've just described -- but it would be handy to add
this post-merge hook script into hooks/ (exec bit removed) such that
on a clone, all one would need to do is chmod +x it.   If that's
possible, I'm clearly missing the steps to enable this.

Thanks,

-- Thomas Adam
--

From: Johannes Schindelin
Date: Monday, July 28, 2008 - 5:40 am

Hi,


No, as you said, anything in .git/ is not meant to be tracked.  Besides, 
if a user would change the executable bit (as advised), Git would always 
show this file as modified, making the tree permanently dirty (or worse, 
it could be accidentally be committed as executable).

For all those reasons, it is better to just commit an executable script in 
your <toplevel>/githooks/post-merge and ask your users to copy it to 
.git/hooks/.

Hth,
Dscho

--

From: Thomas Adam
Date: Monday, July 28, 2008 - 5:40 am

Grand.  Thank you very much.  :)

-- Thomas Adam
--

Previous thread: [PATCH] Allow installing in the traditional way by Junio C Hamano on Monday, July 28, 2008 - 12:09 am. (2 messages)

Next thread: [PATCH] Updated the Documentation/config.txt core.autocrlf configuration setting documentation by Jurko Gospodnetić on Monday, July 28, 2008 - 4:48 am. (1 message)