Merge lp:~cjwatson/launchpad/queue-api-fix-non-component-overrides into lp:launchpad
Status: | Merged | ||||
---|---|---|---|---|---|
Approved by: | Steve Kowalik | ||||
Approved revision: | no longer in the source branch. | ||||
Merged at revision: | 15608 | ||||
Proposed branch: | lp:~cjwatson/launchpad/queue-api-fix-non-component-overrides | ||||
Merge into: | lp:launchpad | ||||
Diff against target: |
38 lines (+15/-2) 2 files modified
lib/lp/soyuz/model/queue.py (+4/-2) lib/lp/soyuz/tests/test_packageupload.py (+11/-0) |
||||
To merge this branch: | bzr merge lp:~cjwatson/launchpad/queue-api-fix-non-component-overrides | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Steve Kowalik (community) | code | Approve | |
Review via email: mp+114236@code.launchpad.net |
Commit message
Fix PackageUpload.
Description of the change
== Summary ==
PackageUpload.
== Proposed fix ==
This is a simple glitch in the handling of changing the archive when the source component changes, and just needs an if guard. There was already a test for this in the PCJ case, but not in the case of an ordinary upload; I cloned-and-hacked the PCJ test.
== LOC Rationale ==
+13. This is part of removing the queue tool, which will get me ~1000 lines of net credit.
== Tests ==
bin/test -vvct PackageUploadTe
== Demo and Q/A ==
Using lp:ubuntu-archive-tools, run something like 'queue -l dogfood -x some-section override some-source-