Merge lp:~jtv/launchpad/bug-832661 into lp:launchpad
Status: | Superseded |
---|---|
Proposed branch: | lp:~jtv/launchpad/bug-832661 |
Merge into: | lp:launchpad |
Diff against target: |
654 lines (+323/-98) 9 files modified
lib/lp/soyuz/doc/publishing.txt (+9/-15) lib/lp/soyuz/interfaces/distributionjob.py (+5/-2) lib/lp/soyuz/model/archive.py (+3/-6) lib/lp/soyuz/model/distroseriesdifferencejob.py (+17/-2) lib/lp/soyuz/model/publishing.py (+68/-72) lib/lp/soyuz/tests/test_distroseriesdifferencejob.py (+119/-1) lib/lp/soyuz/tests/test_publishing.py (+70/-0) lib/lp/testing/factory.py (+18/-0) lib/lp/testing/tests/test_factory.py (+14/-0) |
To merge this branch: | bzr merge lp:~jtv/launchpad/bug-832661 |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Launchpad code reviewers | Pending | ||
Review via email: mp+73009@code.launchpad.net |
This proposal has been superseded by a proposal from 2011-08-26.
Commit message
Create DSDJs from requestDeletion.
Description of the change
= Summary =
When we mark a SourcePackagePu
== Proposed fix ==
Move some code around and then leave the real problem — making it scale — for someone else.
== Pre-implementation notes ==
Julian pointed out Archive.is_main; it's likely to be cheaper than checking DistroSeries.
== Implementation details ==
The core of it is IDistroSeriesDi
== Tests ==
{{{
./bin/test -vvc lp.soyuz.
./bin/test -vvc lp.soyuz.
}}}
== Demo and Q/A ==
Deleting an archive should still mark all its source publication records as deleted.
= Launchpad lint =
The lint is actually remaining pre-existing lint from files I only touched in a previous branch (and fixed in yet another branch), both of which I've been unable to land because of bug 833743 and various other buildbot failures. It's been a wild few days. I'll have to re-submit this branch to register the dependencies on those other branches, so the lint reported here will no longer apply.
Checking for conflicts and issues in changed files.
Linting changed files:
lib/lp/
lib/lp/
lib/lp/
lib/lp/
lib/lp/
lib/lp/
lib/lp/
lib/lp/
lib/lp/
./lib/lp/
119: want exceeds 78 characters.
367: want exceeds 78 characters.
925: want exceeds 78 characters.
927: want exceeds 78 characters.
929: want exceeds 78 characters.
952: want exceeds 78 characters.
./lib/lp/
350: redefinition of function 'private' from line 346