From: Steve Reinhardt <steve.reinhardt@amd.com>
Date: Tue, 15 Oct 2013 22:22:41 +0000 (-0400)
Subject: scons: fix minor update-ref bug in regressions
X-Git-Tag: stable_2014_02_15~81
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=fe61a074a2563e7e5d7d57f0b0ce2158f651e883;p=gem5.git

scons: fix minor update-ref bug in regressions

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.
---

diff --git a/tests/SConscript b/tests/SConscript
index 8e4b42d02..5c1eb5f7b 100644
--- a/tests/SConscript
+++ b/tests/SConscript
@@ -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)