Hi,

On Wed, 14 Dec 2011, Jonas Smedegaard wrote:
> On 11-12-14 at 08:17am, Raphael Hertzog wrote:
> > On Tue, 13 Dec 2011, Simon Ruderich wrote:
> > > The attached patch fixes this and passes DEB_BUILD_MAINT_OPTIONS to 
> > > dpkg-buildflags, but I'm not sure if it's the best way to do it.
> > 
> > The attached patch is not enough. There are other variables that are 
> > important.
> > 
> > Please check /usr/share/dpkg/buildflags.mk as an example of how this 
> > should be done.
> > 
> > Better, I suggest that you rely on dpkg-dev doing the work for you and 
> > that you just include this Makefile and use the standard variables.
> 
> Thanks for your attention to this.
> 
> I am unsure what you suggest above - do you suggest that CDBS not mess 
> with variables at all, or that the proposed patch not be used at all?

I suggest that the patch be reworked to not redo what
/usr/share/dpkg/buildflags.mk already does correctly and to find a way
to reuse /usr/share/dpkg/buildflags.mk.

Unfortunately the interface of /usr/share/dpkg/buildflags.mk means that
CFLAGS/CXXFLAGS/CPPFLAGS/LDFLAGS/FFLAGS are directly modified so you if
you want to store their values in other variables, you might have to backup
the values before including that Makefile and restore them afterwards.

(Of course, if you find this too messy, feel free to find another way to
nicely resolve the problem :-))

Cheers,
-- 
Raphaël Hertzog ◈ Debian Developer

Pre-order a copy of the Debian Administrator's Handbook and help
liberate it: http://debian-handbook.info/liberation/



--
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