[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