generate-contents-files.py ignores the release pocket
Bug #771727 reported by
William Grant
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Launchpad itself |
Fix Released
|
High
|
Jeroen T. Vermeulen |
Bug Description
generate-
def getPocketSuffix
"""Query the distribution's pocket suffixes."""
return self.queryDistr
But the suffix for Release is '', so split only returns suffixes for non-Release pockets.
Related branches
lp://staging/~jtv/launchpad/db-bug-771727
- Robert Collins (community): Approve
-
Diff: 61 lines (+14/-5)2 files modifiedlib/lp/archivepublisher/scripts/generate_contents_files.py (+2/-5)
lib/lp/archivepublisher/tests/test_generate_contents_files.py (+12/-0)
tags: | added: derivation |
Changed in launchpad: | |
status: | Triaged → In Progress |
tags: |
added: qa-ok removed: qa-needstesting |
Changed in launchpad: | |
status: | Fix Committed → Fix Released |
To post a comment you must log in.
As it turns out, it's not the split() that does it. LpQueryDistro deliberately doesn't even return the empty pocket suffix.
Disaster strikes in this bit of shell script that I converted incorrectly:
# Combine all supported series with all pocket suffixes to build the list "${new_ suites: +$new_suites }$suite" ubuntu/ dists/$ suite$pocket_ suffix" ]; then suites= "$new_suites $suite$ pocket_ suffix" "$new_suites"
# of suites for which we will generate/update Contents.gz.
new_suites=
for suite in $suites; do
new_suites=
for pocket_suffix in $pocket_suffixes; do
if [ -d "$ARCHIVE/
new_
fi
done
done
suites=
This adds the $suite to $new_suites, and then for each non-empty pocket suffix adds $suite$ pocket_ suffix to $new_suites.