Code review comment for lp:~thumper/launchpad/lp-distroseries

Revision history for this message
Tim Penhey (thumper) wrote :

On Thu, 12 Nov 2009 18:32:17 Jonathan Lange wrote:
> Review: Needs Fixing
> Thanks for fixing this Tim. The fix is good, and I like the new name for
> makeDistroRelease.
>
> However, I think the error message will be bogus if you try
> lp://dev/ubuntu/warty-backports. Could you please add a test for this?

It says:

$ bzr revno lp://dev/ubuntu/warty-backports
bzr: ERROR: Invalid url supplied to transport: "lp://dev/ubuntu/warty-
backports": Warty is a distroseries, and a distroseries doesn't have a default
branch.

It is still valid, but doesn't show the relationship between "Warty" and
"warty-backports". Suggestions?

« Back to merge proposal