[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