Thomas Heijligen has uploaded this change for review.

View Change

Makefile: unify the look of TARGET_* prints

Change-Id: I69959c20aa2e43ed67b3057c37e964a34cdab136
Signed-off-by: Thomas Heijligen <thomas.heijligen@secunet.com>
---
M Makefile
1 file changed, 6 insertions(+), 8 deletions(-)

git pull ssh://review.coreboot.org:29418/flashrom refs/changes/17/58617/1

diff --git a/Makefile b/Makefile
index c09e9cf..4120546 100644
--- a/Makefile
+++ b/Makefile
@@ -895,17 +895,15 @@
 	@if [ $(CC_WORKING) = yes ]; \
 		then echo $$($(CC) --version 2>/dev/null | head -1); \
 		else echo no; echo Aborting.; exit 1; fi
-	@echo Target arch is $(ARCH)
+	@echo "Target arch: $(ARCH)"
 	@if [ $(ARCH) = unknown ]; then echo Aborting.; exit 1; fi
-	@echo Target OS is $(TARGET_OS)
+	@echo "Target OS: $(TARGET_OS)"
 	@if [ $(TARGET_OS) = unknown ]; then echo Aborting.; exit 1; fi
-	@echo Target endian is $(ENDIAN)
+	@if [ $(TARGET_OS) = libpayload ] && $(CC) --version 2>&1 | grep -q -v coreboot; then \
+		echo "  Warning: It seems you are not using coreboot's reference compiler."; \
+		echo "  This might work but usually does not, please beware."; fi
+	@echo "Target endian: $(ENDIAN)"
 	@if [ $(ENDIAN) = unknown ]; then echo Aborting.; exit 1; fi
-ifeq ($(TARGET_OS), libpayload)
-	@$(CC) --version 2>&1 | grep -q coreboot || \
-		( echo "Warning: It seems you are not using coreboot's reference compiler."; \
-		  echo "This might work but usually does not, please beware." )
-endif
 
 hwlibs: compiler
 	@printf "" > .libdeps

To view, visit change 58617. To unsubscribe, or for help writing mail filters, visit settings.