[Build-common-hackers] Bug#712729: The fix for this breaks other packages

Felipe Sateler fsateler at debian.org
Wed Sep 3 18:47:58 UTC 2014

Control: reopen -1
Control: severity -1 serious

On Thu, 21 Aug 2014 22:04:00 +0000 Jonas Smedegaard <dr at jones.dk> wrote:
> Source: cdbs
> Source-Version: 0.4.126
>    * Fix preserve compiler flags set using DEB_* environment variables,
>      in langcore.mk.
>      Closes: bug#712729. Thanks to Simon Ruderich, Emilio Pozuelo Monfort
>      and Markus.

This change introduces a bug: packages with names that are illegal
shell variable names. For an example, see pulseaudio[1]. This change
results in build flags not being applied.

The filter needs to be smarter.Perhaps only exporting the variables
that look like the ones in the buildflags manpage should be exported:

allvars = $(filter DEB_%,$(.VARIABLES))
filt = $(foreach suf,APPEND PREPEND SET STRIP OPTIONS,$(filter
exp = $(foreach flag,$(filt),$(flag)="$($(flag))")

Please fix this asap, as this change is breaking other packages.

[1] https://buildd.debian.org/status/fetch.php?pkg=pulseaudio&arch=kfreebsd-i386&ver=5.0-11&stamp=1409110310

More information about the Build-common-hackers mailing list