From 7f43ab90972ef7e2b015af583dc0e3d78d3e35c2 Mon Sep 17 00:00:00 2001
From: Mark Travis <mtravis@ripple.com>
Date: Tue, 23 Sep 2014 16:52:11 -0700
Subject: [PATCH] Improvements to SConstruct: * Default target is release
 instead of debug (scons with no arguments). * All targets now include debug
 symbols, including release. Rationale: "out of the box" builds of rippled
 using plain "scons" or "scons -j4" will produce a debug instead of a release
 build, which could underperform.

---
 SConstruct | 9 +++------
 1 file changed, 3 insertions(+), 6 deletions(-)

diff --git a/SConstruct b/SConstruct
index 5085f470c8..598802071f 100644
--- a/SConstruct
+++ b/SConstruct
@@ -259,6 +259,7 @@ def config_env(toolchain, variant, env):
             '-Wno-sign-compare',
             '-Wno-char-subscripts',
             '-Wno-format',
+            '-g' # enable debug symbols for gcc and clang
             ])
 
         if toolchain == 'clang':
@@ -323,11 +324,7 @@ def config_env(toolchain, variant, env):
             '-rdynamic'
             ])
 
-        if variant == 'debug':
-            env.Append(CCFLAGS=[
-                '-g'
-                ])
-        elif variant == 'release':
+        if variant == 'release':
             env.Append(CCFLAGS=[
                 '-O3',
                 '-fno-strict-aliasing'
@@ -505,7 +502,7 @@ else:
         default_toolchain = 'clang'
     else:
         raise ValueError("Don't understand toolchains in " + str(toolchains))
-default_variant = 'debug'
+default_variant = 'release'
 default_target = None
 
 for source in [