launchpad: remove headers from Contents files
Bug #1638219 reported by
Paul Wise (Debian)
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Launchpad itself |
Fix Released
|
Low
|
Colin Watson | ||
Bug Description
Some of the Contents files (but not all) have a plain text header in
them with documentation explaining how to use the files.
It isn't very useful to include plain text documentation inside files
that are machine-readable. Such documentation should appear in the doc/
directory instead if it is to be documented within the repo itself.
Removing the header will fix this apt-file bug immediately,
without having to update apt-file in all Ubuntu/Debian suites:
https:/
The Debian ftp-masters recently made the same change in dak:
https:/
affects launchpad
subscribe ubuntu-archive
--
bye,
pabs
Related branches
lp:~cjwatson/launchpad/remove-contents-header
- William Grant: Approve (code)
-
Diff: 139 lines (+2/-70)4 files modifiedcronscripts/publishing/gen-contents/Contents.top (+0/-32)
cronscripts/publishing/gen-contents/apt_conf_header.template (+0/-2)
lib/lp/archivepublisher/scripts/generate_contents_files.py (+2/-22)
lib/lp/archivepublisher/tests/test_generate_contents_files.py (+0/-14)
| tags: | added: api lp-soyuz soyuz-publish trivial |
| Changed in launchpad: | |
| assignee: | nobody → Colin Watson (cjwatson) |
| status: | New → In Progress |
| importance: | Undecided → Low |
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-ok removed: qa-needstesting |
Revision history for this message
| Colin Watson (cjwatson) wrote | #2 |
| Changed in launchpad: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
Fixed in stable r18249 <http://