Merge lp:~thumper/launchpad/lp-distroseries into lp:launchpad
Proposed by
Tim Penhey
Status: | Merged |
---|---|
Approved by: | Tim Penhey |
Approved revision: | no longer in the source branch. |
Merged at revision: | 9870 |
Proposed branch: | lp:~thumper/launchpad/lp-distroseries |
Merge into: | lp:launchpad |
Diff against target: |
68 lines (+17/-1) 4 files modified
lib/canonical/launchpad/xmlrpc/faults.py (+1/-1) lib/lp/code/interfaces/linkedbranch.py (+5/-0) lib/lp/code/xmlrpc/tests/test_branch.py (+8/-0) lib/lp/testing/factory.py (+3/-0) |
To merge this branch: | bzr merge lp:~thumper/launchpad/lp-distroseries |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Jonathan Lange (community) | Approve | ||
Review via email: mp+14777@code.launchpad.net |
Commit message
Don't oops if resolving a distroseries for lp: traversal.
To post a comment you must log in.
A relatively simple branch that addresses bug 480024.
tests:
TestExpandURL
QA: ubuntu/ warty
bzr revno lp:ubuntu/karmic
or
bzr revno lp://dev/
Gets a nice error like: ubuntu/ warty ubuntu/ warty": Warty is a distroseries, and a distroseries doesn't have a default branch.
$ bzr revno lp://dev/
bzr: ERROR: Invalid url supplied to transport: "lp://dev/
instead of an oops.