SCons: Separately label the global non-sticky options.
authorGabe Black <gblack@eecs.umich.edu>
Tue, 1 Mar 2011 11:00:42 +0000 (03:00 -0800)
committerGabe Black <gblack@eecs.umich.edu>
Tue, 1 Mar 2011 11:00:42 +0000 (03:00 -0800)
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.

SConstruct

index 9bfd1d1c8ce58314051099339aad42f8ec7e455e..d33f5082ef0bf3fec8cffdbe3661287e6401d4de 100755 (executable)
@@ -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)