Package: hello-debhelper
Version: 2.2-3
Severity: wishlist
Tags: patch
User: ubuntu-de...@lists.ubuntu.com
Usertags: origin-ubuntu ubuntu-patch karmic

hello-debhelper fails to cross-build, for much the same reasons as
hello: that is, it doesn't set the proper configure arguments, and it
forces CC to a compiler for the build architecture.

Given that hello-debhelper already requires debhelper 7, I think it
might be nice to showcase the new dh_auto_* commands, which get the
configure arguments right. The following patch does this.

diff -u hello-debhelper-2.2/debian/rules hello-debhelper-2.2/debian/rules
--- hello-debhelper-2.2/debian/rules
+++ hello-debhelper-2.2/debian/rules
@@ -2,7 +2,6 @@
 
 package = hello-debhelper
 
-CC = gcc
 CFLAGS = -g -Wall
 
 ifeq (,$(findstring noopt,$(DEB_BUILD_OPTIONS)))
@@ -10,18 +9,17 @@
 endif
 
 clean:
-       dh_clean
        rm -f build
-       [ ! -f Makefile ] || $(MAKE) distclean
+       dh_auto_clean
+       dh_clean
 
 install: build
        dh_clean
-       dh_installdirs
-       $(MAKE) prefix=$(CURDIR)/debian/$(package)/usr install
+       dh_auto_install
 
 build:
-       ./configure --prefix=/usr
-       $(MAKE) CC="$(CC)" CFLAGS="$(CFLAGS)"
+       dh_auto_configure
+       dh_auto_build -- CFLAGS="$(CFLAGS)"
        touch build
 
 binary-indep: install

Thanks,

-- 
Colin Watson                                       [cjwat...@ubuntu.com]



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org

Reply via email to