[MERGE] Move use_existing and create_prefix all the way down to clone_on_transport, reducing duplicate work.

Robert Collins robert.collins at canonical.com
Thu Apr 23 02:31:18 BST 2009


On Thu, 2009-04-23 at 11:04 +1000, Andrew Bennetts wrote:
> Robert Collins wrote:
> > This isn't a dramatic change, but it does cut one round trip off
> > creating new branches - we were creating the directory and then trying
> > again, just in case it hadn't worked.
> 
> bb:approve
> 
> Looks reasonable to me.  I'm a bit ambivalent about raising BzrCommandError
> from non-UI code, but I don't think that's a particularly important issue
> here.  Otherwise pushing this logic down looks good.

Thanks. I've further refactorings which push that back higher up ;).
Still, - landing this.

-Rob
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: This is a digitally signed message part
Url : https://lists.ubuntu.com/archives/bazaar/attachments/20090423/ae056a75/attachment.pgp 


More information about the bazaar mailing list