[PATCH 3 of 3] hook: report untrusted hooks as failure (issue5110) (BC)

Pierre-Yves David pierre-yves.david at ens-lyon.org
Fri Apr 15 15:20:33 EDT 2016



On 04/15/2016 12:12 PM, timeless wrote:
> can i convince you to replace "denied" with "skipped"?

Skipped seemed a bit too light for the abort case:

Abort: pretxnopen.foo hook skipped (from unstrusted source)

(it is not really "skipped" as it aborted the whole operation).

What do you think?

Cheers,

-- 
Pierre-Yves David


More information about the Mercurial-devel mailing list