scons: Fix the regression tests.
authorGabe Black <gabeblack@google.com>
Wed, 18 Oct 2017 23:48:17 +0000 (16:48 -0700)
committerGabe Black <gabeblack@google.com>
Thu, 19 Oct 2017 00:41:14 +0000 (00:41 +0000)
A recent scons cleanup introduced a small bug when setting the variant path of
the regression test SConscript. When turned into a relative path, the
directory can be interpreted differently depending on what it's considered
relative to. When left as a scons node, there is no ambiguity.

Change-Id: I162afdc4fcef81db096ec8304f81bc98fa6bf899
Reviewed-on: https://gem5-review.googlesource.com/5221
Reviewed-by: Jason Lowe-Power <jason@lowepower.com>
Maintainer: Gabe Black <gabeblack@google.com>

src/SConscript

index 911e5a37c31e7cd009cb2fea9b0617f7c6f1095e..76bf8d1b46538f028db816587f6ed7815765f61a 100755 (executable)
@@ -1117,7 +1117,7 @@ def makeEnv(env, label, objsfx, strip=False, disable_partial=False, **kwargs):
 
     # Set up regression tests.
     SConscript(os.path.join(env.root.abspath, 'tests', 'SConscript'),
-               variant_dir=Dir('tests').Dir(new_env.Label).path,
+               variant_dir=Dir('tests').Dir(new_env.Label),
                exports={ 'env' : new_env }, duplicate=False)
 
 # Start out with the compiler flags common to all compilers,