[PATCH] Use compilation flags.
dirk at hohndel.org
Thu Aug 21 03:50:26 PDT 2014
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?
> 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)
More information about the subsurface