From 461b831d9786da29f9ea0d376ee7185ee5df2de9 Mon Sep 17 00:00:00 2001
From: Sebastian Andrzej Siewior <sebastian@breakpoint.cc>
Date: Sat, 28 Mar 2009 23:00:32 +0100
Subject: [PATCH] Shift selection of debug-level

Currently configure switches on gcc to debug mode what means -O0. Adding
-d none does not help because it is filtered out earlier. -d bla would
help if bla were an allowed keyword.
This patch simply shifts the "compiler is gcc" block to the left so -O2 is
selected if the user selected -d none on the command line. Other compilers
remain with no extra switches.
---
 wscript | 41 ++++++++++++++++++++---------------------
 1 file changed, 20 insertions(+), 21 deletions(-)

diff --git a/wscript b/wscript
index c5555412..4ee9d820 100644
--- a/wscript
+++ b/wscript
@@ -181,28 +181,27 @@ def configure (conf):
             debug_level = 'debug'
         else:
             debug_level = 'none'
-    if debug_level != 'none':
-        if compiler == 'gcc':
-            if debug_level == 'debug':
-                conf.env.append_value ('CCFLAGS', '-Wall -O0 -g'.split ())
-            elif debug_level == 'full':
-                # -Wdeclaration-after-statement
-                # -Wmissing-declarations -Wmissing-prototypes
-                # -Wwrite-strings
-                conf.env.append_value ('CCFLAGS',
-                    '-Wall -Wextra -O1 -g '
-                    '-Waggregate-return -Wno-unused-parameter '
-                    '-Wno-missing-field-initializers '
-                    '-Wunsafe-loop-optimizations '
-                    '-Wredundant-decls -Wmissing-noreturn '
-                    '-Wshadow -Wpointer-arith -Wcast-align '
-                    '-Winline -Wformat-security '
-                    '-Winit-self -Wmissing-include-dirs -Wundef '
-                    '-Wmissing-format-attribute -Wnested-externs '
-                    '-DG_ENABLE_DEBUG'.split ())
-            else:
-                conf.env.append_value ('CCFLAGS', '-O2')
+    if compiler == 'gcc':
+        if debug_level == 'debug':
+            conf.env.append_value ('CCFLAGS', '-Wall -O0 -g'.split ())
+        elif debug_level == 'full':
+            # -Wdeclaration-after-statement
+            # -Wmissing-declarations -Wmissing-prototypes
+            # -Wwrite-strings
+            conf.env.append_value ('CCFLAGS',
+                '-Wall -Wextra -O1 -g '
+                '-Waggregate-return -Wno-unused-parameter '
+                '-Wno-missing-field-initializers '
+                '-Wunsafe-loop-optimizations '
+                '-Wredundant-decls -Wmissing-noreturn '
+                '-Wshadow -Wpointer-arith -Wcast-align '
+                '-Winline -Wformat-security '
+                '-Winit-self -Wmissing-include-dirs -Wundef '
+                '-Wmissing-format-attribute -Wnested-externs '
+                '-DG_ENABLE_DEBUG'.split ())
         else:
+            conf.env.append_value ('CCFLAGS', '-O2')
+    elif debug_level != 'none':
             Utils.pprint ('RED', 'No debugging level support for ' + compiler)
             sys.exit (1)