Merge lp:~statik/python-distutils-extra/modernize into lp:python-distutils-extra

Proposed by Elliot Murphy
Status: Merged
Merged at revision: not available
Proposed branch: lp:~statik/python-distutils-extra/modernize
Merge into: lp:python-distutils-extra
Diff against target: 40 lines (+4/-5)
1 file modified
debian/local/python-mkdebian (+4/-5)
To merge this branch: bzr merge lp:~statik/python-distutils-extra/modernize
Reviewer Review Type Date Requested Status
Martin Pitt (community) Needs Fixing
Review via email: mp+18726@code.launchpad.net

Commit message

Some updates to python-mkdebian.

To post a comment you must log in.
Revision history for this message
Elliot Murphy (statik) wrote :

This branch changes some of the defaults in python-mkdebian. Maybe you will hate this idea, maybe it's good. I know defaults can be controversial, but I believe these changes are good defaults for someone who doesn't know a lot about debian packaging who just wants to turn a python module into a debian package.

I changed from python-central to python-support for consistency with debian python packaging team preference for python-support, to make it easier to move packages that are created with this tool upstream into debian.

I move from CDBS to DH7 because I really like what Joey has done in dh7 and how simple the overrides are. For context, look at slide 44 and 45 of http://kitenet.net/~joey/talks/debhelper/debhelper-slides.odp

I pass --buildsystem=python_distutils so that packages work even if there is a makefile in the root (I and some other upstreams like having a makefile with a check command even for a python project). I am not 100% confident whether dh7 takes care of everything that /usr/share/cdbs/1/rules/langpack.mk was handling.

Update to compat level 7.

Update to Standards-Version: 3.8.3

Finally I fix a typo that caused additional_dependencies defaulting to None instead of []

Revision history for this message
Martin Pitt (pitti) wrote :

Elliot,

thanks for the update. Some remarks:

 * I deliberately used DH 6, so that people could use quickly and p-d-e on Hardy (which does not have DH 7 yet). Doing this move is fine after lucid released and is out in the wild for a while.

 * If we ever get applications which got packaged that way into Ubuntu main, we need to change the packaging back to cdbs, since dh7 knows nothing about our i18n infrastructure (such as adding the gettext domain to .desktop files, stripping translations from gconf schemas, and so on). So for now I'd prefer to stick with CDBS.

I'm fine with the move to pysupport, and of course bumping the standards-version. Also, thanks for the typo fix.

review: Needs Fixing
160. By Elliot Murphy

fixes per review. stick with CDBS due to I18N support. stick with source format 1.0 and compat level 6 to enable this to be used with quickly on Hardy for now.

Revision history for this message
Elliot Murphy (statik) wrote :

Thanks for the quick review! Updated per your comments, also dropped the change to make packages source format 3.0 (quilt) because of what you said about using this on hardy for a while.

161. By Elliot Murphy

Only depend on debhelper 6.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'debian/local/python-mkdebian'
--- debian/local/python-mkdebian 2010-02-05 05:13:13 +0000
+++ debian/local/python-mkdebian 2010-02-05 22:45:21 +0000
@@ -56,8 +56,7 @@
56 if not os.path.exists('debian/rules'):56 if not os.path.exists('debian/rules'):
57 f = open('debian/rules', 'w')57 f = open('debian/rules', 'w')
58 f.write('''#!/usr/bin/make -f58 f.write('''#!/usr/bin/make -f
5959DEB_PYTHON_SYSTEM := pysupport
60DEB_PYTHON_SYSTEM := pycentral
6160
62include /usr/share/cdbs/1/rules/debhelper.mk61include /usr/share/cdbs/1/rules/debhelper.mk
63include /usr/share/cdbs/1/class/python-distutils.mk62include /usr/share/cdbs/1/class/python-distutils.mk
@@ -140,7 +139,7 @@
140 'Build-Depends': '''cdbs (>= 0.4.43),139 'Build-Depends': '''cdbs (>= 0.4.43),
141 debhelper (>= 6),140 debhelper (>= 6),
142 python,141 python,
143 python-central (>= 0.6.11),142 python-support (>= 0.6.4),
144 python-distutils-extra (>= 2.10)%s''' % bdeps,143 python-distutils-extra (>= 2.10)%s''' % bdeps,
145 'Maintainer': author,144 'Maintainer': author,
146 'Package': egg['Name'],145 'Package': egg['Name'],
@@ -154,7 +153,7 @@
154Priority: extra153Priority: extra
155Build-Depends: %(Build-Depends)s154Build-Depends: %(Build-Depends)s
156Maintainer: %(Maintainer)s155Maintainer: %(Maintainer)s
157Standards-Version: 3.8.2156Standards-Version: 3.8.3
158XS-Python-Version: current157XS-Python-Version: current
159158
160Package: %(Package)s159Package: %(Package)s
@@ -282,7 +281,7 @@
282 help='Add string changelog to debian/changelog (can be specified multiple times)')281 help='Add string changelog to debian/changelog (can be specified multiple times)')
283parser.add_option('', '--dependency', dest='dependencies', action='append',282parser.add_option('', '--dependency', dest='dependencies', action='append',
284 help='Add additional debian package dependency (can be specified multiple times)')283 help='Add additional debian package dependency (can be specified multiple times)')
285parser.set_defaults(changelog=None, dependency=[])284parser.set_defaults(changelog=None, dependencies=[])
286options, args = parser.parse_args()285options, args = parser.parse_args()
287286
288egg = get_egg_info()287egg = get_egg_info()

Subscribers

People subscribed via source and target branches

to all changes:
to status/vote changes: