Re: StGit hooks

From: Andreas Ericsson <ae@op5.se>
Date: Nov 28, 2007, 13:14 UTC
Message-ID: <474D69A7.6020404@op5.se>
In-Reply-To: <20071128121905.GA15953@diana.vm.bytemark.co.uk>

Karl Hasselström wrote:

Show 19 quoted lines

> On 2007-11-28 12:44:02 +0100, Andreas Ericsson wrote:
> 
>> Karl Hasselström wrote:
>>
>>> Also, if StGit is to set up hooks automatically (commit hooks,
>>> pre-rebase hooks, whatever), it'd be nice to not have to worry
>>> about overwriting any existing hooks the user might have. But git
>>> currently allows only one hook script per hook, right?
>> Yes, but you can obviously call any number of scripts and programs
>> from within the hook that git executes.
> 
> That doesn't help here, however, since the user and not StGit "owns"
> the "top-level" hook. StGit would have to rely on the user having
> installed a specific kind of multiplexer as a hook script (e.g. one
> that executes everything under .git/hooks/$hook.d/). Or it would have
> to install it itself, and hope that moving any existing hook to the
> subdirectory where the multiplexer looks for hooks doesn't break
> anything. Both solutions are problematic.
>

The user-defined hook can be kept in the hooks directory too. It just needs to be named in such a way that git will never have a hook named like that. For that reason, I think it would be easiest to just agree for the git core to never call any hooks prefixed with "stgit" or some such. I think the odds for it happening by chance are remote, to say the least.

---%<---%<--- #!/bin/sh

do_stgit_things

sh -c '"$GIT_DIR/hooks/pre-stgit-$action-hook" "$@"' exit $? ---%<---%<---

-- 
Andreas Ericsson                   andreas.ericsson@op5.se
OP5 AB                             www.op5.se
Tel: +46 8-230225                  Fax: +46 8-230231