Code review comment for lp:~maxb/udd/local-branches

Revision history for this message
James Westby (james-w) wrote :

On Sun, 03 Oct 2010 20:35:54 -0000, Max Bowsher <email address hidden> wrote:
> Max Bowsher has proposed merging lp:~maxb/udd/local-branches into lp:udd.
>
> Requested reviews:
> Ubuntu Distributed Development Developers (udd)
>
>
> When investigating a failure like the "NoSuchTag" failures, a key
> problem is that it is impossible to experiment with added tags to
> validate that they will fix the conversion.

>
> Therefore, I've written a new option --local-branches - when used, a
> special BranchStore (thanks for having that factored out! :-) ) is
> used, which pushes and pulls from
> BASE_DIR/localbranches/PACKAGENAME/DISTRO-SERIES by default, but if
> that doesn't exist, it first populates it from Launchpad.

Nice idea. I'll review tomorrow, thanks.

Just to note that I've been looking at NoSuchTag today, and think I have
a handle on it, check the referenced bug.

Sorry if I've caused duplicate effort.

Thanks,

James

« Back to merge proposal