[ANNOUNCE] bzr shell hooks plugin
Ian Clatworthy
ian.clatworthy at internode.on.net
Wed Jan 30 07:39:13 GMT 2008
Jelmer Vernooij wrote:
> Hi,
>
> I've just published a simple plugin for Bazaar that allows specifying
> shell scripts as hooks.
>
> The bzr branch is available at
> http://people.samba.org/bzr/jelmer/bzr-shell-hooks/trunk
Hooray! Here are a few comments:
* the example in README says "echo" when you mean "foo.sh" I think
* we ought to trap OSError explicitly for missing commands
* maybe look in multiple config files, not just branch.conf?
* we need to support multiple command line scripts per hook per
config file using the Hg approach of optional tagging, e.g.
[hooks]
pre_commit = foo.sh
pre_commit.also = bar.sh
This has been on my TODO list for a few weeks now. Any objections if I
migrate this functionality into the core and add these changes? (I'm
happy to make the core implementation forwardly compatible with your
code as best I can.)
Ian C.
More information about the bazaar
mailing list