Merge lp:~wgrant/launchpad/bug-433739-debug-archive-urls into lp:launchpad

Proposed by William Grant
Status: Merged
Merged at revision: not available
Proposed branch: lp:~wgrant/launchpad/bug-433739-debug-archive-urls
Merge into: lp:launchpad
Diff against target: 26 lines
2 files modified
lib/lp/soyuz/doc/archive.txt (+4/-0)
lib/lp/soyuz/model/archive.py (+1/-0)
To merge this branch: bzr merge lp:~wgrant/launchpad/bug-433739-debug-archive-urls
Reviewer Review Type Date Requested Status
Julian Edwards (community) Approve
Review via email: mp+13107@code.launchpad.net
To post a comment you must log in.
Revision history for this message
William Grant (wgrant) wrote :

This branch fixes bug #433739 by trivially defining archive_url for archives with purpose DEBUG. The published archive ends up with '-debug' on the end, so the URL should too.

Revision history for this message
Julian Edwards (julian-edwards) wrote :

Thanks for the fix William.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'lib/lp/soyuz/doc/archive.txt'
--- lib/lp/soyuz/doc/archive.txt 2009-09-23 07:41:13 +0000
+++ lib/lp/soyuz/doc/archive.txt 2009-10-09 06:00:40 +0000
@@ -1443,6 +1443,10 @@
1443 ... ubuntutest, ArchivePurpose.PRIMARY).archive_url1443 ... ubuntutest, ArchivePurpose.PRIMARY).archive_url
1444 http://ftpmaster.internal/ubuntutest1444 http://ftpmaster.internal/ubuntutest
14451445
1446 >>> print getUtility(IArchiveSet).getByDistroPurpose(
1447 ... ubuntu, ArchivePurpose.DEBUG).archive_url
1448 http://ftpmaster.internal/ubuntu-debug
1449
1446If the archive is private, the url may be different as private PPAs1450If the archive is private, the url may be different as private PPAs
1447are published to a secure location.1451are published to a secure location.
14481452
14491453
=== modified file 'lib/lp/soyuz/model/archive.py'
--- lib/lp/soyuz/model/archive.py 2009-09-30 14:30:46 +0000
+++ lib/lp/soyuz/model/archive.py 2009-10-09 06:00:40 +0000
@@ -259,6 +259,7 @@
259 archive_postfixes = {259 archive_postfixes = {
260 ArchivePurpose.PRIMARY : '',260 ArchivePurpose.PRIMARY : '',
261 ArchivePurpose.PARTNER : '-partner',261 ArchivePurpose.PARTNER : '-partner',
262 ArchivePurpose.DEBUG : '-debug',
262 }263 }
263264
264 if self.is_ppa:265 if self.is_ppa: