[coreboot-gerrit] Patch set updated for coreboot: util: Update makefiles for junit testing

[Martin Roth (martinroth@google.com)]

Martin Roth (martinroth at google.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/12120

-gerrit

commit 9335cfaebef019b675d48265674f39d8bd9845c0
Author: Martin Roth <martinroth at google.com>
Date:   Wed Oct 21 14:50:00 2015 -0600

    util: Update makefiles for junit testing
    
    - Have clean remove junit.xml files.
    - Remove junit.xml target from cbmem makefile - this is in the top
    level Makefile.inc now.
    - add distclean targets to makefiles.
    - Make sure all makefiles have .PHONY set up.
    - rm commands need -f or they will fail if the file they're trying
    to remove doesn't exist, causing the build to fail.
    
    Change-Id: I2f0635f2c0a9417e3377a90c8d67103323c4a72f
    Signed-off-by: Martin Roth <martinroth at google.com>
---
 util/cbmem/Makefile       | 19 +------------------
 util/ectool/Makefile      |  6 +++++-
 util/inteltool/Makefile   |  2 +-
 util/nvramtool/Makefile   |  2 +-
 util/superiotool/Makefile |  6 ++++--
 util/viatool/Makefile     |  1 +
 6 files changed, 13 insertions(+), 23 deletions(-)

diff --git a/util/cbmem/Makefile b/util/cbmem/Makefile
index 91bb045..41079b0 100644
--- a/util/cbmem/Makefile
+++ b/util/cbmem/Makefile
@@ -31,7 +31,7 @@ all: $(PROGRAM)
 $(PROGRAM): $(OBJS)
 
 clean:
-	rm -f $(PROGRAM) *.o *~
+	rm -f $(PROGRAM) *.o *~ junit.xml
 
 distclean: clean
 	rm -f .dependencies
@@ -43,20 +43,3 @@ distclean: clean
 
 -include .dependencies
 
-junit.xml:
-	echo '<?xml version="1.0" encoding="utf-8"?><testsuite>' > $@.tmp
-	$(MAKE) clean; \
-	echo "<testcase classname='cbmem' name='cbmem'>" >> $@.tmp; \
-	$(MAKE) >> $@.tmp.2 2>&1 && type="system-out" || type="failure"; \
-	if [ $$type = "failure" ]; then \
-		echo "<failure type='buildFailed'>" >> $@.tmp; \
-	else \
-		echo "<$$type>" >> $@.tmp; \
-	fi; \
-	echo '<![CDATA[' >> $@.tmp; \
-	cat $@.tmp.2 >> $@.tmp; \
-	echo "]]></$$type>" >>$@.tmp; \
-	rm -f $@.tmp.2; \
-	echo "</testcase>" >> $@.tmp; \
-	echo "</testsuite>" >> $@.tmp
-	mv $@.tmp $@
diff --git a/util/ectool/Makefile b/util/ectool/Makefile
index 3ab0bc8..6225503 100644
--- a/util/ectool/Makefile
+++ b/util/ectool/Makefile
@@ -38,7 +38,11 @@ install: $(PROGRAM)
 	$(INSTALL) $(PROGRAM) $(PREFIX)/sbin
 
 clean:
-	rm *.o $(PROGRAM)
+	rm -f *.o $(PROGRAM) junit.xml
+
+distclean: clean
 
 %.o: %.c
 	$(CC) $(CFLAGS) -c $^ -I. -o $@
+
+.PHONY: all install clean distclean
diff --git a/util/inteltool/Makefile b/util/inteltool/Makefile
index d5fb1da..3fbcf9c 100644
--- a/util/inteltool/Makefile
+++ b/util/inteltool/Makefile
@@ -45,7 +45,7 @@ $(PROGRAM): $(OBJS)
 	$(CC) $(CFLAGS) -o $(PROGRAM) $(OBJS) $(LDFLAGS)
 
 clean:
-	rm -f $(PROGRAM) *.o *~
+	rm -f $(PROGRAM) *.o *~ junit.xml
 
 distclean: clean
 	rm -f .dependencies
diff --git a/util/nvramtool/Makefile b/util/nvramtool/Makefile
index 1f19733..7df3aa3 100644
--- a/util/nvramtool/Makefile
+++ b/util/nvramtool/Makefile
@@ -54,7 +54,7 @@ $(PROGRAM): $(OBJS)
 	$(STRIP) $(STRIP_ARGS) $(PROGRAM)
 
 clean:
-	rm -f $(PROGRAM) $(OBJS)
+	rm -f $(PROGRAM) $(OBJS) junit.xml
 
 distclean: clean
 	rm -f .dependencies
diff --git a/util/superiotool/Makefile b/util/superiotool/Makefile
index b005a3f..0736713 100644
--- a/util/superiotool/Makefile
+++ b/util/superiotool/Makefile
@@ -73,6 +73,8 @@ install: $(PROGRAM)
 	$(INSTALL) -p -m644 $(PROGRAM).8 $(DESTDIR)$(PREFIX)/share/man/man8
 
 clean:
-	rm -f $(PROGRAM) *.o
+	rm -f $(PROGRAM) *.o junit.xml
 
-.PHONY: all install clean
+distclean: clean
+
+.PHONY: all install clean distclean
diff --git a/util/viatool/Makefile b/util/viatool/Makefile
index 1658557..f1ac3a0 100644
--- a/util/viatool/Makefile
+++ b/util/viatool/Makefile
@@ -55,6 +55,7 @@ clean:
 	rm -f $(PROGRAM) $(OBJS)
 	# Remove backup files created by some editors
 	find ./ |grep *~ |xargs rm -f
+	rm -f junit.xml
 
 distclean: clean
 	rm -f .dependencies