Bobby R. Bruce has submitted this change. (
https://gem5-review.googlesource.com/c/public/gem5/+/32097 )
Change subject: util,tests: Added exit code to the compiler tests
......................................................................
util,tests: Added exit code to the compiler tests
This testing script should return a non-exit code when one of the
compilations fail.
Change-Id: Ie15bc5779372dd31d784eaffdee4b04abb9a1b11
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/32097
Reviewed-by: Ciro Santilli <[email protected]>
Reviewed-by: Hoa Nguyen <[email protected]>
Reviewed-by: Daniel Carvalho <[email protected]>
Maintainer: Bobby R. Bruce <[email protected]>
Tested-by: kokoro <[email protected]>
---
M util/compiler-tests.sh
1 file changed, 5 insertions(+), 0 deletions(-)
Approvals:
Ciro Santilli: Looks good to me, approved
Daniel Carvalho: Looks good to me, approved
Hoa Nguyen: Looks good to me, approved
Bobby R. Bruce: Looks good to me, approved
kokoro: Regressions pass
diff --git a/util/compiler-tests.sh b/util/compiler-tests.sh
index a574fd3..cd47bd9 100755
--- a/util/compiler-tests.sh
+++ b/util/compiler-tests.sh
@@ -74,6 +74,8 @@
touch "${exits}"
echo "compiler,build_target,exit_code" >> "${exits}"
+exit_code=0 # We return a non-zero exit code if any of the compilations
fail.
+
for compiler in ${images[@]}; do
echo "Starting build tests with '${compiler}'..."
# Generate a randomized list of build targets
@@ -121,6 +123,7 @@
echo "${compiler},${build}/gem5${build_opt},${result}"
"${exits}"
if [ ${result} -ne 0 ]; then
+ exit_code=1
echo " ! Failed with exit code ${result}."
else
echo " Done."
@@ -130,3 +133,5 @@
done
mv "${test_dir}" "${test_dir_final}"
+
+exit ${exit_code}
--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/32097
To unsubscribe, or for help writing mail filters, visit
https://gem5-review.googlesource.com/settings
Gerrit-Project: public/gem5
Gerrit-Branch: develop
Gerrit-Change-Id: Ie15bc5779372dd31d784eaffdee4b04abb9a1b11
Gerrit-Change-Number: 32097
Gerrit-PatchSet: 2
Gerrit-Owner: Bobby R. Bruce <[email protected]>
Gerrit-Reviewer: Bobby R. Bruce <[email protected]>
Gerrit-Reviewer: Ciro Santilli <[email protected]>
Gerrit-Reviewer: Daniel Carvalho <[email protected]>
Gerrit-Reviewer: Giacomo Travaglini <[email protected]>
Gerrit-Reviewer: Hoa Nguyen <[email protected]>
Gerrit-Reviewer: kokoro <[email protected]>
Gerrit-MessageType: merged
_______________________________________________
gem5-dev mailing list -- [email protected]
To unsubscribe send an email to [email protected]
%(web_page_url)slistinfo%(cgiext)s/%(_internal_name)s
