[MERGE] add hook to inspect and modify url for fallback location of a branch

Andrew Bennetts andrew.bennetts at canonical.com
Fri Oct 17 08:11:17 BST 2008


Andrew Bennetts has voted tweak.
Status is now: Semi-approved
Comment:
This seems ok, but I think it would be good to be explicit about what 
happens if multiple hook functions are registered for 
transform_fallback_location.  Most of our existing hooks are mostly 
passive, so this hook is breaking new ground.

So, I think I'd like to see docs and tests addressing what happens when 
there are multiple hook functions.  Perhaps just document it explicitly 
as "undefined" behaviour for now (on the basis that anything more is a 
YAGNI)?


For details, see: 
http://bundlebuggy.aaronbentley.com/project/bzr/request/%3C48ED5C67.9020408%40canonical.com%3E
Project: Bazaar



More information about the bazaar mailing list