Split ftpmaster helper classes in individual files
Bug #75621 reported by
Celso Providelo
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Launchpad itself |
Fix Released
|
Low
|
Colin Watson | ||
Bug Description
canonical/
{{{
__all__ = [
'ArchiveOve
'ArchiveOve
'ArchiveCru
'ArchiveCru
'PubSourceC
'ChrootMana
'ChrootMana
'SyncSource',
]
}}}
They don't share any contexts and doesn't look like they will in a short-term.
Each class should be moved to an individual file.
Related branches
lp:~cjwatson/launchpad/split-ftpmaster
- j.c.sackett (community): Approve
- Richard Harding (community): Approve (code*)
-
Diff: 2170 lines (+950/-914)21 files modifiedlib/lp/archivepublisher/scripts/generate_contents_files.py (+2/-2)
lib/lp/archivepublisher/scripts/publish_ftpmaster.py (+2/-2)
lib/lp/scripts/utilities/importfascist.py (+2/-2)
lib/lp/soyuz/doc/manage-chroot.txt (+6/-4)
lib/lp/soyuz/scripts/chrootmanager.py (+230/-0)
lib/lp/soyuz/scripts/ftpmaster.py (+0/-868)
lib/lp/soyuz/scripts/obsolete_distroseries.py (+101/-0)
lib/lp/soyuz/scripts/packageremover.py (+106/-0)
lib/lp/soyuz/scripts/pubsourcechecker.py (+214/-0)
lib/lp/soyuz/scripts/querydistro.py (+260/-0)
lib/lp/soyuz/scripts/tests/test_chrootmanager.py (+2/-2)
lib/lp/soyuz/scripts/tests/test_lpquerydistro.py (+2/-2)
lib/lp/soyuz/scripts/tests/test_obsoletedistroseries.py (+3/-5)
lib/lp/soyuz/scripts/tests/test_overrides_checker.py (+2/-2)
lib/lp/soyuz/scripts/tests/test_populatearchive.py (+3/-5)
lib/lp/soyuz/scripts/tests/test_removepackage.py (+5/-7)
scripts/ftpmaster-tools/archive-override-check.py (+2/-2)
scripts/ftpmaster-tools/lp-query-distro.py (+2/-2)
scripts/ftpmaster-tools/lp-remove-package.py (+2/-3)
scripts/ftpmaster-tools/manage-chroot.py (+2/-3)
scripts/ftpmaster-tools/obsolete-distroseries.py (+2/-3)
| Changed in soyuz: | |
| importance: | Undecided → Wishlist |
| status: | Unconfirmed → Confirmed |
| visibility: | private → public |
| Changed in launchpad: | |
| assignee: | nobody → Colin Watson (cjwatson) |
| status: | Triaged → In Progress |
Revision history for this message
| Launchpad QA Bot (lpqabot) wrote | #1 |
| tags: | added: qa-needstesting |
| Changed in launchpad: | |
| status: | In Progress → Fix Committed |
| tags: |
added: qa-untestable removed: qa-needstesting |
| Changed in launchpad: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
Fixed in stable r14703 <http://