Merge lp:~cjwatson/launchpad/remove-deb-ar-assumption into lp:launchpad
Proposed by
Colin Watson
Status: | Merged |
---|---|
Merged at revision: | 18974 |
Proposed branch: | lp:~cjwatson/launchpad/remove-deb-ar-assumption |
Merge into: | lp:launchpad |
Prerequisite: | lp:~cjwatson/launchpad/write-file-bytes |
Diff against target: |
259 lines (+119/-50) 2 files modified
lib/lp/archiveuploader/nascentuploadfile.py (+20/-35) lib/lp/archiveuploader/tests/test_nascentuploadfile.py (+99/-15) |
To merge this branch: | bzr merge lp:~cjwatson/launchpad/remove-deb-ar-assumption |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
William Grant | code | Approve | |
Review via email: mp+367917@code.launchpad.net |
Commit message
Check the .deb format using dpkg-deb rather than ar.
Description of the change
Debian has been talking about changing the outer .deb container file format to something other than ar, e.g. in order to lift size limits. It may or may not go anywhere, but it reminded me that we really shouldn't be making any particular assumptions about what the container format is: the interface we should rely on is that provided by dpkg.
To post a comment you must log in.