From: Gabe Black <gblack@eecs.umich.edu>
Date: Tue, 1 Mar 2011 11:00:42 +0000 (-0800)
Subject: SCons: Separately label the global non-sticky options.
X-Git-Tag: stable_2012_02_02~506
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=9e608a6ad7a965a08798ba25822b5da7cd17d7a0;p=gem5.git

SCons: Separately label the global non-sticky options.

The global sticky options were being printed with a heading, and then the
global nonsticky options were being printed immediately after them without a
heading. Because the two lists ran together and the first had its own heading,
it looked like -all- those options where sticky even though some of them
aren't. This change adds a label to the second list so it's clear they're
different.
---

diff --git a/SConstruct b/SConstruct
index 9bfd1d1c8..d33f5082e 100755
--- a/SConstruct
+++ b/SConstruct
@@ -338,20 +338,23 @@ global_nonsticky_vars.AddVariables(
     ('update_ref', 'Update test reference outputs', False)
     )
 
+# Update main environment with values from ARGUMENTS & global_sticky_vars_file
+global_sticky_vars.Update(main)
+global_nonsticky_vars.Update(main)
+global_help_texts = {
+    "global_sticky" : global_sticky_vars.GenerateHelpText(main),
+    "global_nonsticky" : global_nonsticky_vars.GenerateHelpText(main)
+}
 
 # base help text
 help_text = '''
 Usage: scons [scons options] [build options] [target(s)]
 
 Global sticky options:
-'''
-
-# Update main environment with values from ARGUMENTS & global_sticky_vars_file
-global_sticky_vars.Update(main)
-global_nonsticky_vars.Update(main)
-
-help_text += global_sticky_vars.GenerateHelpText(main)
-help_text += global_nonsticky_vars.GenerateHelpText(main)
+%(global_sticky)s
+Global nonsticky options:
+%(global_nonsticky)s
+''' % global_help_texts
 
 # Save sticky variable settings back to current variables file
 global_sticky_vars.Save(global_sticky_vars_file, main)