Package: hdparm
Version: 9.42-1
Severity: normal
Tags: patch
User: [email protected]
Usertags: cross

hdparm fails to cross-build, because it tries to use the native compiler
rather than the cross-compiler.

It turns out that cdbs has support for setting CC appropriately.
However, hdparm defeats this by setting DEB_MAKE_ENVVARS too early, so
cdbs sees that it already has a value and doesn't change it.  The
simplest fix appears to be to append to DEB_MAKE_ENVVARS *after*
including the cdbs Makefile snippets, rather than setting it beforehand.
After doing this, hdparm cross-builds cleanly.  Patch follows.

  * Set DEB_MAKE_ENVVARS differently in order to take advantage of
    cross-building support in cdbs.

diff -u hdparm-9.42/debian/rules hdparm-9.42/debian/rules
--- hdparm-9.42/debian/rules
+++ hdparm-9.42/debian/rules
@@ -22,11 +22,11 @@
 DEB_UPDATE_RCD_PARAMS := 'start 07 S . '
 DEB_FIXPERMS_EXCLUDE := 20hdparm
 DEB_DH_INSTALLUDEV_ARGS := --priority=85
-DEB_MAKE_ENVVARS := STRIP="true" LDFLAGS=
 
 include /usr/share/cdbs/1/rules/debhelper.mk
 include /usr/share/cdbs/1/class/makefile.mk
 
+DEB_MAKE_ENVVARS += STRIP="true" LDFLAGS=
 DEB_MAKE_INSTALL_TARGET := install DESTDIR=$(CURDIR)/debian/tmp
 
 common-install-prehook-arch::

Thanks,

-- 
Colin Watson                                       [[email protected]]


-- 
To UNSUBSCRIBE, email to [email protected]
with a subject of "unsubscribe". Trouble? Contact [email protected]

Reply via email to