diff options
author | Keith Short <keithshort@chromium.org> | 2019-07-11 17:54:57 -0600 |
---|---|---|
committer | Martin Roth <martinroth@google.com> | 2019-07-13 18:33:51 +0000 |
commit | 31af70dd96406c596306f424ae929664e7862fc2 (patch) | |
tree | 8407289c1a1d76856b2cc4e495a4d324b0de9037 /payloads/libpayload | |
parent | 19398245b498e2ced39f3828f804c94bb82e9a14 (diff) |
util/testing: Ensure coreboot-gerrit fails if libpayload build fails
The JUnit output from the libpayload builds was getting deleted by the
coreinfo build. Move the libpayload to later in the coreboot-gerrit
job.
Also add messages to stdout indicating the various libpayload configs
that are built and a message indicating when all libpayload builds are
complete.
BUG=b:137380189
TEST=Upload test commit that includes a libpayload compile error and
verify buildbot fails.
Change-Id: I43b55f402216582dcf81be34171437be345572ab
Signed-off-by: Keith Short <keithshort@chromium.org>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/34183
Reviewed-by: Patrick Georgi <pgeorgi@google.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'payloads/libpayload')
-rw-r--r-- | payloads/libpayload/Makefile.inc | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/payloads/libpayload/Makefile.inc b/payloads/libpayload/Makefile.inc index 7787762b43..1fa07f9a7a 100644 --- a/payloads/libpayload/Makefile.inc +++ b/payloads/libpayload/Makefile.inc @@ -130,6 +130,7 @@ junit.xml: echo '<?xml version="1.0" encoding="utf-8"?><testsuite>' > $@.tmp for i in $(filter-out %.old,$(wildcard configs/*)); do \ $(MAKE) clean; \ + echo "Building libpayload for $$i"; \ cp "$$i" junit_config; \ $(MAKE) olddefconfig DOTCONFIG=junit_config V=$(V) Q=$(Q) 2>/dev/null >/dev/null; \ echo "<testcase classname='libpayload' name='$$i'>" >> $@.tmp; \ @@ -146,6 +147,7 @@ junit.xml: echo "</testcase>" >> $@.tmp; \ done echo "</testsuite>" >> $@.tmp + echo "libpayload build complete, test results in $@" mv $@.tmp $@ test-configs: |