Merge lp:~evgeni/libappindicator/use-autoreconf.mk into lp:~ubuntu-desktop/libappindicator/ubuntu

Proposed by Evgeni Golov
Status: Merged
Merged at revision: 296
Proposed branch: lp:~evgeni/libappindicator/use-autoreconf.mk
Merge into: lp:~ubuntu-desktop/libappindicator/ubuntu
Diff against target: 24 lines (+2/-2)
1 file modified
debian/rules (+2/-2)
To merge this branch: bzr merge lp:~evgeni/libappindicator/use-autoreconf.mk
Reviewer Review Type Date Requested Status
Robert Ancell Approve
Ken VanDine Pending
Review via email: mp+106750@code.launchpad.net

Description of the change

  use autoreconf.mk instead of calling dh_autoreconf by hand

  - fixes FTBFS when there are multiple Python versions
    (dh_autoreconf would be called multiple times in this case, which isn't
     allowed)
  - properly cleans autoreconf'ed files in the clean target
  - patchsys-quilt.mk needs to be included *before* autoreconf.mk or
    debhelper will leave uncleaned files in debian/, leading to
    E: libappindicator source: temporary-debhelper-file
       python-appindicator.debhelper.log and 9 others
    from lintian

To post a comment you must log in.
Revision history for this message
Robert Ancell (robert-ancell) wrote :

Looks good, uploaded to quantal.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'debian/rules'
2--- debian/rules 2012-03-09 20:47:45 +0000
3+++ debian/rules 2012-05-22 07:28:22 +0000
4@@ -8,10 +8,11 @@
5 export MONO_SHARED_DIR=$(CURDIR)
6 export CSC=/usr/bin/mono-csc
7
8+include /usr/share/cdbs/1/rules/patchsys-quilt.mk
9+include /usr/share/cdbs/1/rules/autoreconf.mk
10 include /usr/share/cdbs/1/rules/debhelper.mk
11 include /usr/share/cdbs/1/class/gnome.mk
12 include /usr/share/cdbs/1/rules/utils.mk
13-include /usr/share/cdbs/1/rules/patchsys-quilt.mk
14
15 API_VERSION = 0.1
16
17@@ -20,7 +21,6 @@
18 DEB_CONFIGURE_EXTRA_FLAGS += --disable-scrollkeeper --enable-gtk-doc --enable-introspection
19
20 configure-stamp-%:
21- dh_autoreconf
22 mkdir build-$*
23 cd build-$* && PYTHON=`which $*` \
24 $(DEB_CONFIGURE_SCRIPT) $(DEB_CONFIGURE_NORMAL_ARGS) --srcdir=..

Subscribers

People subscribed via source and target branches

to all changes: