tests: log_call is not returning any value
authorGiacomo Travaglini <giacomo.travaglini@arm.com>
Fri, 15 May 2020 09:31:51 +0000 (10:31 +0100)
committerGiacomo Travaglini <giacomo.travaglini@arm.com>
Thu, 18 Jun 2020 14:28:14 +0000 (14:28 +0000)
JIRA: https://gem5.atlassian.net/projects/GEM5/issues/GEM5-533

Change-Id: I2713ddacc762d614e3992718ea234287d06c179a
Signed-off-by: Giacomo Travaglini <giacomo.travaglini@arm.com>
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/30214
Maintainer: Bobby R. Bruce <bbruce@ucdavis.edu>
Tested-by: kokoro <noreply+kokoro@google.com>
Reviewed-by: Hoa Nguyen <hoanguyen@ucdavis.edu>
tests/gem5/suite.py

index 4cf0f81a25e3d08e2dcf432e5f0a32dbf04f00bf..329e31fc7d853a18bf82b344a40c6bbc31f49d50 100644 (file)
@@ -178,7 +178,6 @@ def _create_test_run_gem5(config, config_args, gem5_args):
         # I.E. Only the returncode verifier will use the gem5_returncode
         # fixture, but we always require it even if that verifier isn't being
         # ran.
-        returncode = fixtures[constants.gem5_returncode_fixture_name]
         tempdir = fixtures[constants.tempdir_fixture_name].path
         gem5 = fixtures[constants.gem5_binary_fixture_name].path
         command = [
@@ -191,7 +190,6 @@ def _create_test_run_gem5(config, config_args, gem5_args):
         command.append(config)
         # Config_args should set up the program args.
         command.extend(config_args)
-        returncode.value = log_call(params.log, command, stdout=sys.stdout,
-                                                         stderr=sys.stderr)
+        log_call(params.log, command, stdout=sys.stdout, stderr=sys.stderr)
 
     return test_run_gem5