Merge lp:~mvo/python-distutils-extra/mvo into lp:python-distutils-extra

Proposed by Michael Vogt
Status: Merged
Merged at revision: 274
Proposed branch: lp:~mvo/python-distutils-extra/mvo
Merge into: lp:python-distutils-extra
Diff against target: 54 lines (+13/-16)
1 file modified
debian/local/python-mkdebian (+13/-16)
To merge this branch: bzr merge lp:~mvo/python-distutils-extra/mvo
Reviewer Review Type Date Requested Status
Martin Pitt (community) Needs Fixing
Review via email: mp+87357@code.launchpad.net

Description of the change

Update to use dh_python2, dh7 and a more current dep5 format.

Note that this will break compatbility with lucid targets but should work from maverick onwards.

To post a comment you must log in.
Revision history for this message
Martin Pitt (pitti) wrote :

Thanks! Fine for me, pysupport dropped the /opt stuff anyway, so dh_python2 needs to grow support for this. Didier ack'ed this.

This needs to add back the conditional support for "--with langpack", presumably with a file test.

review: Needs Fixing

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'debian/local/python-mkdebian'
2--- debian/local/python-mkdebian 2011-11-08 15:51:04 +0000
3+++ debian/local/python-mkdebian 2012-01-03 14:01:26 +0000
4@@ -55,28 +55,26 @@
5
6 if not os.path.exists('debian/compat'):
7 f = open('debian/compat', 'w')
8- f.write('6\n')
9+ f.write('8\n')
10 f.close()
11
12 prefix_string = ''
13 if prefix:
14- prefix_string = "DEB_PYTHON_PREFIX_ARG := %s" % prefix
15+ prefix_string = '''override_dh_auto_install:
16+ dh_auto_install -- --install-scripts=%s \
17+ --install-lib=%s
18+
19+override_dh_python2:
20+ dh_python2 %s
21+''' % (prefix, prefix, prefix)
22+
23 if not os.path.exists('debian/rules') or force_rules:
24 f = open('debian/rules', 'w')
25 f.write('''#!/usr/bin/make -f
26-DEB_PYTHON_SYSTEM := pysupport
27+%%:
28+ dh $@ --with python2
29+
30 %s
31-
32-include /usr/share/cdbs/1/rules/debhelper.mk
33-include /usr/share/cdbs/1/class/python-distutils.mk
34-# langpack.mk is relevant on Ubuntu only, not Debian; it does not matter if it's missing
35--include /usr/share/cdbs/1/rules/langpack.mk
36-
37-common-binary-post-install-arch::
38- -dh_icons -a
39-
40-common-binary-post-install-indep::
41- -dh_icons -i
42 ''' % prefix_string)
43 f.close()
44 os.chmod('debian/rules', 0755)
45@@ -158,8 +156,7 @@
46 'Source': egg['Name'],
47 'Build-Depends': '''cdbs (>= 0.4.43),
48 debhelper (>= 6),
49- python,
50- python-support (>= 0.6.4),
51+ python (>= 2.6.6-3~),
52 python-distutils-extra (>= 2.10)%s''' % bdeps,
53 'Maintainer': author,
54 'Package': egg['Name'],

Subscribers

People subscribed via source and target branches

to all changes:
to status/vote changes: