BLD: add "coverage" as scons build mode.

scons `build` variable can now be set to "coverage".
Remove the unnecessary scons variable "coverage".

This closes #10.

Author: pavoljuhas

SConstruct

@@ -65,7 +65,7 @@ vars.Add(PathVariable(
 vars.Add(EnumVariable(
     'build',
     'compiler settings',
-    'fast', allowed_values=('debug', 'fast')))
+    'fast', allowed_values=('fast', 'debug', 'coverage')))
 vars.Add(EnumVariable(
     'tool',
     'C++ compiler toolkit to be used',
@@ -76,9 +76,6 @@ vars.Add(BoolVariable(
 vars.Add(BoolVariable(
     'profile',
     'build with profiling information', False))
-vars.Add(BoolVariable(
-    'coverage',
-    'build for code coverage analysis', False))
 vars.Add(
     'tests',
     'fixed-string patterns for selecting unit tests', None)
@@ -91,7 +88,6 @@ env.Help(MY_SCONS_HELP % vars.GenerateHelpText(env))
 env['has_objcryst'] = None
 btags = [env['build'], platform.machine()]
 if env['profile']:  btags.append('profile')
-if env['coverage']:  btags.append('coverage')
 builddir = env.Dir('build/' + '-'.join(btags))
 
 Export('env')

src/SConscript

@@ -44,6 +44,9 @@ else:
 # Configure build variants
 if env['build'] == 'debug':
     env.Append(CCFLAGS='-g')
+elif env['build'] == 'coverage':
+    env.Append(CCFLAGS=['-g', '--coverage', '-O0'])
+    env.Append(LINKFLAGS='--coverage')
 elif env['build'] == 'fast':
     env.AppendUnique(CCFLAGS=['-O3'] + fast_optimflags)
     env.AppendUnique(CPPDEFINES={'NDEBUG' : None})
@@ -53,9 +56,6 @@ if env['profile']:
     env.AppendUnique(CCFLAGS='-pg')
     env.AppendUnique(LINKFLAGS='-pg')
 
-if env['coverage']:
-    env.Append(CCFLAGS=['--coverage', '-O0'])
-    env.Append(LINKFLAGS='--coverage')
 
 # configure boost and ObjCryst libraries unless non-relevant.
 skip_configure = (GetOption('clean') or GetOption('help') or