Make CPU_MODELS a sticky build option.
authorSteve Reinhardt <stever@eecs.umich.edu>
Wed, 29 Mar 2006 03:55:08 +0000 (22:55 -0500)
committerSteve Reinhardt <stever@eecs.umich.edu>
Wed, 29 Mar 2006 03:55:08 +0000 (22:55 -0500)
This causes a crash if you're using scons 0.96.1 *and* you specify
more than one CPU model.  Since the .isa scanner now works with 0.96.91
then upgrading should not be an issue.  For now we're only using one CPU
model (SimpleCPU) so there isn't even a pressing need to upgrade yet.

build/SConstruct:
    Make CPU_MODELS a sticky option.
    This causes a crash if you're using scons 0.96.1 *and* you specify
    more than one CPU model.  Since the .isa scanner now works with 0.96.91
    then upgrading should not be an issue.  For now we're only using one CPU
    model (SimpleCPU) so there isn't even a pressing need to upgrade yet.

--HG--
extra : convert_revision : d8319c4cd5c937c2c033270cef850d19b805d256

build/SConstruct

index 306d3a9dc5e479a4e64383b990f5cbe69225f28b..e1854b1d3034424262b0e01197f05beab44393ca 100644 (file)
@@ -231,6 +231,11 @@ sticky_opts = Options(args=ARGUMENTS)
 sticky_opts.AddOptions(
     EnumOption('TARGET_ISA', 'Target ISA', 'alpha', env['ALL_ISA_LIST']),
     BoolOption('FULL_SYSTEM', 'Full-system support', False),
+    # There's a bug in scons 0.96.1 that causes ListOptions with list
+    # values (more than one value) not to be able to be restored from
+    # a saved option file.  If this causes trouble then upgrade to
+    # scons 0.96.90 or later.
+    ListOption('CPU_MODELS', 'CPU models', 'all', env['ALL_CPU_LIST']),
     BoolOption('ALPHA_TLASER',
                'Model Alpha TurboLaser platform (vs. Tsunami)', False),
     BoolOption('NO_FAST_ALLOC', 'Disable fast object allocator', False),
@@ -254,12 +259,6 @@ sticky_opts.AddOptions(
 # Non-sticky options only apply to the current build.
 nonsticky_opts = Options(args=ARGUMENTS)
 nonsticky_opts.AddOptions(
-    # This really should be a sticky option, but there's a bug in
-    # scons 0.96.1 that causes ListOptions not to be able to be
-    # restored from a saved option file.  It looks like this is fixed
-    # in 0.96.9, but there's a different bug in that version that means we
-    # can't just upgrade.
-    ListOption('CPU_MODELS', 'CPU models', 'all', env['ALL_CPU_LIST']),
     BoolOption('update_ref', 'Update test reference outputs', False)
     )