Launchpad itself

Merge lp://staging/~wgrant/launchpad/bug-522800-getFileByName into lp://staging/launchpad

bug-522800-getFileByName
Merge into devel

Proposed by William Grant on 2010-12-09

Status:

Merged

Approved by:

Julian Edwards on 2010-12-09

Approved revision:

no longer in the source branch.

Merged at revision:

12043

Proposed branch:

lp://staging/~wgrant/launchpad/bug-522800-getFileByName

Merge into:

lp://staging/launchpad

Diff against target:

332 lines (+113/-140)

5 files modified

lib/lp/soyuz/doc/build-files.txt (+7/-136)
lib/lp/soyuz/model/archive.py (+1/-0)
lib/lp/soyuz/stories/ppa/xx-ppa-files.txt (+2/-2)
lib/lp/soyuz/tests/test_archive.py (+100/-0)
lib/lp/testing/factory.py (+3/-2)

To merge this branch:

bzr merge lp://staging/~wgrant/launchpad/bug-522800-getFileByName

Related bugs:

Bug #522800: Broken link to .orig.tar.gz in PPA package details page (404)	High	Fix Released
Bug #687662: Upload processor attempts to verify hashes against expired files	High	Fix Released

Link a bug report

Reviewer	Review Type	Date Requested	Status
Julian Edwards (community)		2010-12-09	Approve on 2010-12-10
Review via email: mp+43159@code.staging.launchpad.net

Commit message

Archive.getFileByName now returns only unexpired files.

Description of the change

Archive.getFileByName currently doesn't care if the file it returns is expired. That isn't excellent, since an expired file is just about useless. This branch fixes it to only return unexpired files, fixing bug #522800. It also happens to fix bug #687662, a 10.12 regression that was just discovered.

Revision history for this message