[Xen-changelog] [xen-unstable] Do not print `***' when compiling unless we fall over

[Xen patchbot-unstable <patchbot-unstable@xxxxxxxxxxxxxxxxxxx>]

# HG changeset patch
# User Keir Fraser <keir.fraser@xxxxxxxxxx>
# Date 1231755432 0
# Node ID 59d511c4a8d8ba451afc6ebd88e049fa2addf9f5
# Parent  bd78714b8594f5c86a78e5d5cd16879487bbd048
Do not print `***' when compiling unless we fall over

This has finally bugged me enough to do something about it:

make prints `***' when it stops due to an error.  With make -j, and/or
with automated build systems, this error marker can be in the middle
of a large logfile.  It is useful to be able to search for it.

Therefore change occurrences of `*' to `=' for information and warning
messages.  `***' is reserved for cases where the build stops.

A corresponding change is being applied to qemu-xen-unstable.

Signed-off-by: Ian Jackson <ian.jackson@xxxxxxxxxxxxx>
---
 buildconfigs/mk.linux-2.6-common |    4 ++--
 docs/check_pkgs                  |   12 ++++++------
 tools/blktap/drivers/Makefile    |    2 +-
 tools/firmware/Makefile          |    4 ++--
 tools/vtpm/Makefile              |    2 +-
 5 files changed, 12 insertions(+), 12 deletions(-)

diff -r bd78714b8594 -r 59d511c4a8d8 buildconfigs/mk.linux-2.6-common
--- a/buildconfigs/mk.linux-2.6-common  Mon Jan 12 10:16:05 2009 +0000
+++ b/buildconfigs/mk.linux-2.6-common  Mon Jan 12 10:17:12 2009 +0000
@@ -100,10 +100,10 @@ endif
 endif
        $(__NONINT_CONFIG) $(MAKE) -C $(LINUX_SRCDIR) ARCH=$(LINUX_ARCH) 
oldconfig O=$$(/bin/pwd)/$(LINUX_DIR)
        @set -e ; if [ ! -f $(LINUX_DIR)/Makefile ] ; then \
-           echo "***********************************"; \
+           echo "==================================="; \
            echo "oldconfig did not create a Makefile"; \
            echo "Generating $(LINUX_DIR)/Makefile   "; \
-           echo "***********************************"; \
+           echo "==================================="; \
            ( echo "# Automatically generated: don't edit"; \
              echo ""; \
              echo "VERSION = 2"; \
diff -r bd78714b8594 -r 59d511c4a8d8 docs/check_pkgs
--- a/docs/check_pkgs   Mon Jan 12 10:16:05 2009 +0000
+++ b/docs/check_pkgs   Mon Jan 12 10:17:12 2009 +0000
@@ -2,12 +2,12 @@ silent_which ()
 silent_which ()
 {
         which $1 1>/dev/null 2>/dev/null || {
-                echo "*************************************************"
-                echo "*************************************************"
-                echo "* WARNING: Package '$1' is required"
-                echo "*          to build Xen documentation"
-                echo "*************************************************"
-                echo "*************************************************"
+                echo "================================================="
+                echo "================================================="
+                echo "= WARNING: Package '$1' is required"
+                echo "=          to build Xen documentation"
+                echo "================================================="
+                echo "================================================="
         }
         which $1 1>/dev/null 2>/dev/null
 }
diff -r bd78714b8594 -r 59d511c4a8d8 tools/blktap/drivers/Makefile
--- a/tools/blktap/drivers/Makefile     Mon Jan 12 10:16:05 2009 +0000
+++ b/tools/blktap/drivers/Makefile     Mon Jan 12 10:17:12 2009 +0000
@@ -18,7 +18,7 @@ CRYPT_LIB := -lgcrypt
 CRYPT_LIB := -lgcrypt
 else
 CRYPT_LIB := -lcrypto
-$(warning *** libgcrypt not installed: falling back to libcrypto ***)
+$(warning === libgcrypt not installed: falling back to libcrypto ===)
 endif
 
 LDFLAGS_blktapctrl := $(LDFLAGS_libxenctrl) $(LDFLAGS_libxenstore) -L../lib 
-lblktap
diff -r bd78714b8594 -r 59d511c4a8d8 tools/firmware/Makefile
--- a/tools/firmware/Makefile   Mon Jan 12 10:16:05 2009 +0000
+++ b/tools/firmware/Makefile   Mon Jan 12 10:17:12 2009 +0000
@@ -15,10 +15,10 @@ SUBDIRS += hvmloader
 .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 "==========================================================="; \
        echo "Require dev86 package version >= 0.16.14 to build firmware!"; \
        echo "(visit http://www.cix.co.uk/~mayday for more information)"; \
-       echo "***********************************************************"; \
+       echo "==========================================================="; \
        else \
        $(MAKE) subdirs-$@; \
        fi
diff -r bd78714b8594 -r 59d511c4a8d8 tools/vtpm/Makefile
--- a/tools/vtpm/Makefile       Mon Jan 12 10:16:05 2009 +0000
+++ b/tools/vtpm/Makefile       Mon Jan 12 10:17:12 2009 +0000
@@ -89,6 +89,6 @@ build_sub:
                        $(MAKE) -C $(TPM_EMULATOR_DIR); \
                fi \
        else \
-               echo "*** Unable to build VTPMs. libgmp could not be found."; \
+               echo "=== Unable to build VTPMs. libgmp could not be found."; \
        fi
 

_______________________________________________
Xen-changelog mailing list
Xen-changelog@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-changelog