Control: reassign -1 1076932 dpkg-dev Control: forcemerge -1 1076932 Control: affects -1 src:webdis
On Fri, Jul 26, 2024 at 01:20:28PM GMT, Nicolas Boulenguez wrote: > The fact that the previous lazy evaluation mechanism, in which the > $(evals VAR=$(VAR)) trick is already present, did what you expect in > some contexts does not make CFLAGS+= a supported interface, and your > code could break in other contexts. > > Replacing > CFLAGS+= foo > with > DEB_CFLAGS_MAINT_APPEND := foo > in the broken packages seems more fruitful to me, and I would prefer > to help with that. I'm running into a similar problem with webdis (#1076932), however in that case we're doing CFLAGS += $(CPPFLAGS) I'm not sure how to do that with the DEB_CFLAGS_MAINT_APPEND. My naïve attempt at DEB_CFLAGS_MAINT_APPEND = $(CPPFLAGS) results in /usr/share/dpkg/buildflags.mk:51: *** Recursive variable 'dpkg_buildflags_run' references itself (eventually). Stop. Cheers, -- James GPG Key: 4096R/91BF BF4D 6956 BD5D F7B7 2D23 DFE6 91AE 331B A3DB
