From: Robin Gareus <robin@gareus.org>
Date: Sun, 19 Feb 2017 15:01:25 +0000 (+0100)
Subject: Allow to keep environment C/XXFLAGS with debug builds
X-Git-Tag: 5.7~96
X-Git-Url: https://main.carlh.net/gitweb/?p=ardour.git;a=commitdiff_plain;h=0653766f790fc17afe323e616c10bcc21efe4915;ds=sidebyside

Allow to keep environment C/XXFLAGS with debug builds

this is needed for e.g. windows debug builds to pass mstackrealign
---

diff --git a/tools/x-win/compile.sh b/tools/x-win/compile.sh
index f21aec19b1..58350f9e62 100755
--- a/tools/x-win/compile.sh
+++ b/tools/x-win/compile.sh
@@ -67,6 +67,7 @@ export DLLTOOL=${XPREFIX}-dlltool
 CFLAGS="-mstackrealign" \
 CXXFLAGS="-mstackrealign" \
 LDFLAGS="-L${PREFIX}/lib" ./waf configure \
+	--keepflags \
 	--dist-target=mingw \
 	--also-include=${PREFIX}/include \
 	$ARDOURCFG \
diff --git a/wscript b/wscript
index c2dcfd80f3..9222b98d17 100644
--- a/wscript
+++ b/wscript
@@ -716,6 +716,8 @@ def options(opt):
                    help='Build internal libs as static libraries')
     opt.add_option('--use-external-libs', action='store_true', default=False, dest='use_external_libs',
                    help='Use external/system versions of some bundled libraries')
+    opt.add_option('--keepflags', action='store_true', default=False, dest='keepflags',
+                    help='Do not ignore CFLAGS/CXXFLAGS environment vars')
     opt.add_option('--luadoc', action='store_true', default=False, dest='luadoc',
                     help='Compile Tool to dump LuaBindings (needs C++11)')
     opt.add_option('--canvasui', action='store_true', default=False, dest='canvasui',
@@ -808,7 +810,7 @@ def configure(conf):
         conf.env['MSVC_TARGETS'] = ['x64']
         conf.load('msvc')
 
-    if Options.options.debug:
+    if Options.options.debug and not Options.options.keepflags:
         # Nuke user CFLAGS/CXXFLAGS if debug is set (they likely contain -O3, NDEBUG, etc)
         conf.env['CFLAGS'] = []
         conf.env['CXXFLAGS'] = []