Merge lp:~maxb/lpreview-body/fix-package into lp:~launchpad-committers/lpreview-body/packaging
Proposed by
Max Bowsher
Status: | Merged |
---|---|
Approved by: | Gavin Panella |
Approved revision: | 23 |
Merged at revision: | 23 |
Proposed branch: | lp:~maxb/lpreview-body/fix-package |
Merge into: | lp:~launchpad-committers/lpreview-body/packaging |
Diff against target: |
108 lines (+39/-17) 4 files modified
__init__.py (+27/-16) debian/changelog (+10/-0) debian/control (+1/-0) qa_ready.py (+1/-1) |
To merge this branch: | bzr merge lp:~maxb/lpreview-body/fix-package |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Gavin Panella (community) | Approve | ||
Review via email: mp+59137@code.launchpad.net |
Commit message
[r=allenap][bug=766149] New upstream snapshot (Use lazy hook installation with bzr >= 2.4; Correct regexp used for getting revno currently on edge); Add missing XB-Python-Version field to control, fixing incorrect installation path of the Python files.
Description of the change
Fix the package, which is currently totally broken due to installing the Python files in the incorrect directory.
Then, pull in a couple of upstream changes whilst we're rebuilding it anyway.
To post a comment you must log in.
I know almost nothing about the dirty work involved in packaging, but
a search led me to the following:
The use of XB-Python-Version in the binary package paragraphs of
debian/control file has been deprecated and should be removed in the
normal course of package updates. It never achieved sufficient
deployment to support it's intended purpose of managing Python
transitions. This can be adequately accomplished by examining
package dependencies.
http:// www.debian. org/doc/ packaging- manuals/ python- policy/ ch-module_ packages. html
Should you instead be using X-Python-Version?