[MERGE] StackableBranch

Robert Collins robert.collins at canonical.com
Fri Jun 6 15:19:49 BST 2008


On Fri, 2008-06-06 at 00:40 +1000, Ian Clatworthy wrote:

> Here are the outstanding issues/questions I'd like feedback
> from Robert (and others if interested) on:
> 
> 1. BzrBranch7._get_fallback_repository(url) requires that
>    the url be a branch url, i.e. a url of a shared repo
>    without a branch looks like it wil fail. Is that by
>    design? Acceptable for now, or an oversight?

design

> 2. Is my algorithm in InterPackRepo.search_missing_revision_ids
>    ok or too complex? The comment says "Find ghosts:" so I'm
>    wondering whether the "Double query here:" bit of code can
>    be removed if I just collect ghosts and skip the non ghosts?

I'll read it in detail next week.

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


More information about the bazaar mailing list