Bobby Bruce has submitted this change. (
https://gem5-review.googlesource.com/c/public/gem5/+/58609 )
Change subject: tests: Skip "build/ALL/gem5.opt" build test
......................................................................
tests: Skip "build/ALL/gem5.opt" build test
This commit,
https://gem5-review.googlesource.com/c/public/gem5/+/57509, added
'constants.all_compiled_tag' to test the "build/ALL/gem5.opt" target
once incorporated. However, the test_build.py tests automatically parse
these tags to generate build tests. As such the nightly tests fail
due to trying to build a target which has not yet been added to the
project: https://jenkins.gem5.org/job/nightly/183
This patch skips this build. It may be reverted once
"build/ALL/gem5.opt" is available.
Change-Id: Ib3a01113214f0715cd13ee0da4ee387efd403c8f
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/58609
Reviewed-by: Bobby Bruce <bbr...@ucdavis.edu>
Maintainer: Bobby Bruce <bbr...@ucdavis.edu>
Tested-by: kokoro <noreply+kok...@google.com>
---
M tests/gem5/test_build/test_build.py
1 file changed, 26 insertions(+), 1 deletion(-)
Approvals:
Bobby Bruce: Looks good to me, approved; Looks good to me, approved
kokoro: Regressions pass
diff --git a/tests/gem5/test_build/test_build.py
b/tests/gem5/test_build/test_build.py
index 3a6a534..715b129 100644
--- a/tests/gem5/test_build/test_build.py
+++ b/tests/gem5/test_build/test_build.py
@@ -32,9 +32,10 @@
from testlib import *
common_isas = [constants.gcn3_x86_tag, constants.arm_tag,
constants.riscv_tag]
+skipped_isas = {constants.null_tag, constants.all_compiled_tag}
for isa in constants.supported_isas:
- if isa is constants.null_tag: continue
+ if isa in skipped_isas: continue
for variant in constants.supported_variants:
if isa in common_isas:
--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/58609
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: Ib3a01113214f0715cd13ee0da4ee387efd403c8f
Gerrit-Change-Number: 58609
Gerrit-PatchSet: 3
Gerrit-Owner: Bobby Bruce <bbr...@ucdavis.edu>
Gerrit-Reviewer: Bobby Bruce <bbr...@ucdavis.edu>
Gerrit-Reviewer: Jason Lowe-Power <power...@gmail.com>
Gerrit-Reviewer: kokoro <noreply+kok...@google.com>
Gerrit-MessageType: merged
_______________________________________________
gem5-dev mailing list -- gem5-dev@gem5.org
To unsubscribe send an email to gem5-dev-le...@gem5.org
%(web_page_url)slistinfo%(cgiext)s/%(_internal_name)s
