[coreboot] New patch to review for coreboot: a52ee4d abuild: produce valid junit files

Stefan Reinauer (stefan.reinauer@coreboot.org) gerrit at coreboot.org
Fri Dec 7 22:15:24 CET 2012


Stefan Reinauer (stefan.reinauer at coreboot.org) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/1992

-gerrit

commit a52ee4d9eb81b1586489a7531750ea6894f615b5
Author: Stefan Reinauer <reinauer at chromium.org>
Date:   Fri Dec 7 13:13:26 2012 -0800

    abuild: produce valid junit files
    
    If no valid cross compiler is found, the junit file produced
    by abuild is invalid, missing the closing </testcase> tag.
    This breaks proper reporting in Jenkins of our ARM board at
    this moment.
    
    Change-Id: I94bfc7f334d33ceeb53451a7c5125058c1f33bd4
    Signed-off-by: Stefan Reinauer <reinauer at google.com>
---
 util/abuild/abuild | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/util/abuild/abuild b/util/abuild/abuild
index 9591d13..aefb1ea 100755
--- a/util/abuild/abuild
+++ b/util/abuild/abuild
@@ -368,8 +368,9 @@ function build_target
 			xml ""
 			xml "</mainboard>"
 
-			junit " <testcase classname='board' name='$TARCH/$VENDOR/$MAINBOARD' >"
-			junit "<failure type='NoCrossCompiler'>No cross-compiler for $TARCH found</failure>"
+			junit "<testcase classname='board' name='$TARCH/$VENDOR/$MAINBOARD' >"
+			junit " <failure type='NoCrossCompiler'>No cross-compiler for $TARCH found</failure>"
+			junit "</testcase>"
 
 			return 0
 		else




More information about the coreboot mailing list