[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Xen-devel] [PATCH] [build system] make should fail if it cannot build hvmloader



At the moment, if the tools required to build hvmloader are not present, make will continue building the rest of the system and not indicate that an error has occurred.

This patch causes make to stop and print a message saying that some packages are missing on the build machine and need to be installed.

Please consider applying the patch.

Regards,

Alex Zeffertt
# HG changeset patch
# User Alex Zeffertt <alex.zeffertt@xxxxxxxxxxxxx>
# Date 1235477069 0
# Node ID 72a8404eea4035d38129e2b2097fac5613f05f07
# Parent  32b15413749255e0cd518f25d9202759586dcb27
Make build fail if packages required to build hvmloader not present
... rather than just continuing as if nothing's gone wrong.

Also, corrected the message generated by the makefile which says what
packages are required.

Signed-off-by: Alex Zeffertt<alex.zeffertt@xxxxxxxxxxxxx>

diff -r 32b154137492 -r 72a8404eea40 tools/firmware/Makefile
--- a/tools/firmware/Makefile   Thu Feb 12 10:54:17 2009 +0000
+++ b/tools/firmware/Makefile   Tue Feb 24 12:04:29 2009 +0000
@@ -15,13 +15,12 @@
 .PHONY: all
 all:
        @set -e; if [ $$((`( bcc -v 2>&1 | grep version || echo 0.0.0 ) | cut 
-d' ' -f 3 | awk -F. '{ printf "0x%02x%02x%02x", $$1, $$2, $$3}'`)) -lt 
$$((0x00100e)) ] ; then \
-       echo "==========================================================="; \
-       echo "Require dev86 package version >= 0.16.14 to build firmware!"; \
-       echo "(visit http://www.cix.co.uk/~mayday for more information)"; \
-       echo "==========================================================="; \
-       else \
+       echo 
"=================================================================="; \
+       echo "Require bin86 & bcc packages version >= 0.16.14 to build 
firmware!"; \
+       echo 
"=================================================================="; \
+       false ; \
+       fi
        $(MAKE) subdirs-$@; \
-       fi
 
 
 .PHONY: install
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.