Regressions: Fix the regress script when "all" is used.
authorGabe Black <gblack@eecs.umich.edu>
Sun, 5 Feb 2012 09:23:22 +0000 (01:23 -0800)
committerGabe Black <gblack@eecs.umich.edu>
Sun, 5 Feb 2012 09:23:22 +0000 (01:23 -0800)
When the "all" test is specified, the "tests" list should have two elements in
it, "quick" and "long", not a single element "quick,long". The later would be
appropriate as the default for one of the command line options which are split
at commas, but at that point "tests" should already be a list.

util/regress

index 69b11c04514fa8f2c7a90bd47a2a0d4449afb758..c8704381b057b31034a7f4cd096f7fe63c679885 100755 (executable)
@@ -119,7 +119,7 @@ if compile_variants:
 if not tests:
     tests = ['quick']
 elif 'all' in tests:
-    tests = ['quick,long']
+    tests = ['quick', 'long']
 
 # set up test targets for scons, since we don't have any quick SPARC
 # full-system tests exclude it