scons: fix minor update-ref bug in regressions
authorSteve Reinhardt <steve.reinhardt@amd.com>
Tue, 15 Oct 2013 22:22:41 +0000 (18:22 -0400)
committerSteve Reinhardt <steve.reinhardt@amd.com>
Tue, 15 Oct 2013 22:22:41 +0000 (18:22 -0400)
In the unusual case that regressions are run with --update-ref
when there is no existing regression output, scons gets
confused because it depends on stats.txt to trigger the
update, but it has no indication that running the test will
generate the stats.txt file.  (In the typical case where
stats.txt already exists, scons doesn't care about where
it came from.)

It's easy to fix this just by adding the stats.txt file
to the target list for the test action.

tests/SConscript

index 8e4b42d02df7560a3c6b1828500271bb7761a9a3..5c1eb5f7b94d7dbb9d4bd56cbc55d5c3714311f4 100644 (file)
@@ -282,7 +282,7 @@ def test_builder(env, ref_dir):
     new_stats = tgt('stats.txt')
     status_file = tgt('status')
 
-    env.Command([status_file],
+    env.Command([status_file, new_stats],
                 [env.M5Binary, 'run.py', ref_stats],
                 testAction)