[PATCH] post-commit post-revprop-change post-lock post-unlock hooks documentation

Daniel Shahaf d.s at daniel.shahaf.co.il
Sun Apr 27 06:47:38 CDT 2008


Mark E. Hamilton wrote on Thu, 3 Apr 2008 at 14:54 -0600:
> The description for the four post hooks (post-commit, post-revprop-change,
> post-lock, and post-unlock) all say this:
> 
> The output from, and exit value returned by the post-commit hook program are
> ignored.
> 
> However, this is not exactly correct, since if the exit value is set to
> non-zero the stderr output will be marshalled back to the user (similarly to
> the pre-* version of the hook.)

Ping, should/shouldn't this patch be applied?

Daniel

> If this is the desired behavior (and I like
> it, since it helps diagnose hook-script failures,) IMO the documentation
> should say something like this:
> 
> If the post-commit hook program returns a non-zero exit status the commit
> *will not* be aborted, since it has already completed. However, anything
> printed to stderr will be marshalled back to the client, allowing for
> diagnosis of hook failures.
> 
> I've attached a patch that makes this change (with the appropriate textual
> differences for each hook.)
> 
> 
> 




More information about the svnbook-dev mailing list