diff options
| author | Martin Roth <martinroth@google.com> | 2015-10-21 15:28:21 -0600 |
|---|---|---|
| committer | Patrick Georgi <pgeorgi@google.com> | 2015-10-24 10:55:52 +0200 |
| commit | 917acb5656542e7966846f03073d700fc04dcb33 (patch) | |
| tree | 843feaa1347221e91cf530e52922135584630afd | |
| parent | 0be83c09a066119f5fded3bd1344f90a0a98d52c (diff) | |
| download | coreboot-917acb5656542e7966846f03073d700fc04dcb33.tar.xz | |
util: update junit.xml target
- Display what's happening to the console as well as logging to the
junit.xml file.
- Log the clean in the junit.xml file so if it fails it doesn't just
appear to not have run the test.
- Run both clean and distclean (if distclean exists and runs clean,
this still only runs clean once) so that if distclean doesn't exist
the clean still happens. Don't stop the build if the clean step
fails in case there's no distclean in the util makefile.
- Run the util builds multithreaded. This saves a couple of seconds
and helps find dependency issues that might not be seen if building
single-threaded.
Change-Id: If895295c83faba98661b7c925b65fd436e06b834
Signed-off-by: Martin Roth <martinroth@google.com>
Reviewed-on: http://review.coreboot.org/12121
Tested-by: build bot (Jenkins)
Reviewed-by: Patrick Georgi <pgeorgi@google.com>
| -rw-r--r-- | Makefile.inc | 11 |
1 files changed, 8 insertions, 3 deletions
diff --git a/Makefile.inc b/Makefile.inc index 3a0d0bcf0e..6f9d77f08d 100644 --- a/Makefile.inc +++ b/Makefile.inc @@ -682,14 +682,18 @@ $(obj)/board_id: printf $(CONFIG_BOARD_ID_STRING) > $@ junit.xml: + echo "Building $(UTIL)" echo '<?xml version="1.0" encoding="utf-8"?><testsuite>' > $@.tmp - $(MAKE) -C "util/$(UTIL)" distclean > /dev/null 2>&1 echo "<testcase classname='$(UTIL)' name='$(UTIL)'>" >> $@.tmp - $(MAKE) -C "util/$(UTIL)" >> $@.tmp.2 2>&1 && type="system-out" || type="failure"; \ + -$(MAKE) -j $(CPUS) -C "util/$(UTIL)" clean distclean > $@.tmp.2 2>&1 + $(MAKE) -j $(CPUS) -C "util/$(UTIL)" >> $@.tmp.2 2>&1 && type="system-out" || type="failure"; \ + cat $@.tmp.2; \ if [ "$$type" = "failure" ]; then \ echo "<failure type='buildFailed'>" >> $@.tmp; \ + echo "Building $(UTIL) Failed"; \ else \ echo "<$$type>" >> $@.tmp; \ + echo "Building $(UTIL) Succeeded"; \ fi; \ echo '<![CDATA[' >> $@.tmp; \ cat $@.tmp.2 >> $@.tmp; \ @@ -698,6 +702,7 @@ junit.xml: echo "</testcase>" >> $@.tmp echo "</testsuite>" >> $@.tmp mv $@.tmp "util/$(UTIL)/$@" + echo TOOLLIST= \ cbmem \ @@ -710,4 +715,4 @@ what-jenkins-does: util/abuild/abuild -B -J $(if $(JENKINS_NOCCACHE),,-y) -c $(CPUS) -z -p $(JENKINS_PAYLOAD) -x -X $(top)/abuild-chromeos.xml util/abuild/abuild -B -J $(if $(JENKINS_NOCCACHE),,-y) -c $(CPUS) -z -p $(JENKINS_PAYLOAD) (cd payloads/libpayload; unset COREBOOT_BUILD_DIR; $(MAKE) $(if $(JENKINS_NOCCACHE),,CONFIG_LP_CCACHE=y) V=$(V) Q=$(Q) junit.xml) - $(foreach tool, $(TOOLLIST), $(MAKE) V=$(V) Q=$(Q) UTIL="$(tool)" MFLAGS= MAKEFLAGS= junit.xml; ) + $(foreach tool, $(TOOLLIST), $(MAKE) CPUS=$(CPUS) V=$(V) Q=$(Q) UTIL="$(tool)" MFLAGS= MAKEFLAGS= junit.xml; ) |