Merge lp:~mordred/swift/build-on-lucid into lp:~openstack-ubuntu-packagers/swift/diablo

Proposed by Monty Taylor
Status: Merged
Merged at revision: 49
Proposed branch: lp:~mordred/swift/build-on-lucid
Merge into: lp:~openstack-ubuntu-packagers/swift/diablo
Diff against target: 45 lines (+13/-2)
3 files modified
debian/changelog (+6/-0)
debian/control (+2/-1)
debian/rules (+5/-1)
To merge this branch: bzr merge lp:~mordred/swift/build-on-lucid
Reviewer Review Type Date Requested Status
Clint Byrum (community) Approve
Review via email: mp+75802@code.launchpad.net

Description of the change

Puts in a branching condition as to whether or not to add --with python2. Also relaxes the hard depend on 2.6.6-3~

To post a comment you must log in.
Revision history for this message
Clint Byrum (clint-fewbar) wrote :

+1 from me, makes it use dh_python2 when available, but still leaves the package backportable.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'debian/changelog'
--- debian/changelog 2011-09-02 18:10:30 +0000
+++ debian/changelog 2011-09-16 19:48:24 +0000
@@ -1,3 +1,9 @@
1swift (1.4.3-0ubuntu1) oneiric; urgency=low
2
3 * Work around dh_python2 for lucid. (LP: #848971)
4
5 -- Monty Taylor <mordred@inaugust.com> Fri, 16 Sep 2011 15:40:19 -0400
6
1swift (1.4.3~20110902.354-0ubuntu1) oneiric; urgency=low7swift (1.4.3~20110902.354-0ubuntu1) oneiric; urgency=low
28
3 * New upstream release.9 * New upstream release.
410
=== modified file 'debian/control'
--- debian/control 2011-08-02 17:45:09 +0000
+++ debian/control 2011-09-16 19:48:24 +0000
@@ -9,7 +9,8 @@
9 Michael Barton <michael.barton@rackspace.com>,9 Michael Barton <michael.barton@rackspace.com>,
10 Thomas Goirand <zigo@debian.org>10 Thomas Goirand <zigo@debian.org>
11Build-Depends: debhelper (>= 7.0.50~),11Build-Depends: debhelper (>= 7.0.50~),
12 python-all-dev (>= 2.6.6-3~),12 python-all (>= 2.6),
13 python-all-dev (>= 2.6.6-3~) | python-support,
13 python-openssl,14 python-openssl,
14 python-setuptools,15 python-setuptools,
15 python-webob,16 python-webob,
1617
=== modified file 'debian/rules'
--- debian/rules 2011-08-02 17:45:09 +0000
+++ debian/rules 2011-09-16 19:48:24 +0000
@@ -3,8 +3,12 @@
3# Verbose mode3# Verbose mode
4#export DH_VERBOSE=14#export DH_VERBOSE=1
55
6# Lucid does not have dh_python2, but we would like to be able to use this
7# rules file to build on lucid as well. Thus the branching logic.
8WITH_PYTHON2 = $(shell test -f /usr/bin/dh_python2 && echo "--with python2")
9
6%:10%:
7 dh $@ -with python211 dh $@ ${WITH_PYTHON2}
812
9# clean sphinx build output13# clean sphinx build output
10override_dh_clean:14override_dh_clean:

Subscribers

People subscribed via source and target branches