util: Make running m5 util unit tests opt-in.
authorGabe Black <gabe.black@gmail.com>
Fri, 23 Oct 2020 03:01:00 +0000 (20:01 -0700)
committerGabe Black <gabe.black@gmail.com>
Fri, 4 Dec 2020 22:57:35 +0000 (22:57 +0000)
These tests don't run reliably right now for a few reasons, including
problems with QEMU, and apparently inaccurate information from g++-s
--print-sysroot option.

This may be revisited in the future if those problems can be sorted out.
For now, avoid tripping up new people who won't know to (or how to) work
around those sorts of errors.

Change-Id: Ide42e6c6b27159ff146b8495ae568d1fd377f4f4
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/28179
Reviewed-by: Gabe Black <gabe.black@gmail.com>
Maintainer: Gabe Black <gabe.black@gmail.com>
Tested-by: kokoro <noreply+kokoro@google.com>
util/m5/README.md
util/m5/SConstruct

index 691347816e20af5d156364c97889863c1d0ab8f3..941c499f6e4e179e0ab164dddc4d79bdff9f68c9 100644 (file)
@@ -175,7 +175,7 @@ some other prefix corresponding to that host.
 ### SCons command line flags
 
 --debug-build: Compile with the -g option, and -O0.
---no-test-xml: Exclude the test result XML files from the build.
+--run-tests:   Allow the test result XML files to be build targets.
 --verbose:     Show build command lines and full command output.
 
 ## External dependency detection
index 79fb1fa2f5a8a1faf9f82a4bb4013ba90038b529..6f1ca9fad26706440a126aa07461e3287c4bdd5a 100644 (file)
@@ -44,8 +44,8 @@ def abspath(d):
 
 AddOption('--debug-build', dest='debug_build', action='store_true',
           help='Build with debug info, and disable optimizations.')
-AddOption('--no-test-xml', dest='no_tests', action='store_true',
-          help='Omit test output xml files from the build.')
+AddOption('--run-tests', dest='run_tests', action='store_true',
+          help='Enable test output xml files as build targets.')
 AddOption('--verbose', dest='verbose', action='store_true')
 
 # Universal settings.
@@ -118,8 +118,7 @@ def GTest(env, name, *srcs, **kwargs):
     need_qemu_to_run = 'HAVE_QEMU' in env;
 
     # If we can run this test...
-    if (not need_qemu_to_run or env['HAVE_QEMU']) \
-        and not GetOption('no_tests'):
+    if GetOption('run_tests') and (not need_qemu_to_run or env['HAVE_QEMU']):
         # An XML file which holds the results of the test.
         xml = Dir('test').Dir('result').File('%s.xml' % name)
         # The basic command line for the test.