On Thu, Aug 21, 2014 at 10:16:17AM +0200, Salvo 'LtWorf' Tomaselli wrote: > This makes qmake aware of CFLAGS, CXXFLAGS, LDFLAGS, CPPFLAGS.
Thanks for sending your first two patches and welcome to the team! > This is helpful to easily add flags to the compilation (such as > hardening flags). Question to our Qt people... is that what people usually do? I'll admit that I tried to use CFLAGS when wanting to set some debugging macros in the past (simply habit), but I wonder if there's some reason why this isn't there by default? /D > diff --git a/subsurface.pro b/subsurface.pro > index 1d891a1..b6c81f6 100644 > --- a/subsurface.pro > +++ b/subsurface.pro > @@ -340,3 +340,8 @@ include(subsurface-install.pri) > # to build debuggable binaries on Windows, you need something like this > #QMAKE_CFLAGS_RELEASE=$$QMAKE_CFLAGS_DEBUG -O0 -g > #QMAKE_CXXFLAGS_RELEASE=$$QMAKE_CXXFLAGS_DEBUG -O0 -g > + > +QMAKE_CXXFLAGS += $$(CXXFLAGS) > +QMAKE_CFLAGS += $$(CFLAGS) > +QMAKE_LFLAGS += $$(LDFLAGS) > +QMAKE_CPPFLAGS += $$(CPPFLAGS) > -- > 2.1.0 _______________________________________________ subsurface mailing list [email protected] http://lists.hohndel.org/cgi-bin/mailman/listinfo/subsurface
