[PATCH] Use compilation flags.

Dirk Hohndel 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?

/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


More information about the subsurface mailing list