Patch set updated for coreboot: ecd202d Makefile: Check $CC variable returned from xcompile is not empty.

[Andrew Wu]

Andrew Wu ([e-mail]) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/3905

-gerrit

commit ecd202d4dbc3ef88b42c063a2d70e92e5f3dd22a
Author: Andrew Wu 
Date:   Thu Sep 5 00:21:11 2013 +0800

Makefile: Check $CC variable returned from xcompile is not empty.

If xcompile can't find out suitable GCC compiler for i386/armv7, it
    will not set $CC_i386/$CC_armv7 variable. Makefile sets $CC variable
    from xcompile, and will print strange error messages when executing
    $CC program if $CC is empty.

Add checking to avoid this problem. If $CC is empty, also delete
    invalid .xcompile file, so Make can recreate this file next time.

Change-Id: Ia8d481d76ca52f3351cb99f05779d06947161c5d
    Signed-off-by: Andrew Wu 
---
 Makefile                     | 5 +++++
 payloads/libpayload/Makefile | 5 +++++
 2 files changed, 10 insertions(+)

diff --git a/Makefile b/Makefile
index dec32e5..3b4d931 100644
--- a/Makefile
+++ b/Makefile
@@ -137,6 +137,11 @@ AR := $(AR_$(ARCH-y))

CFLAGS += $(CFLAGS_$(ARCH-y))

+ifeq ($(CC),)
+$(shell rm .xcompile)
+$(error no suitable GCC for $(ARCH-y))
+endif
+
 LIBGCC_FILE_NAME := $(shell test -r `$(CC) -print-libgcc-file-name` && \
    	      $(CC) -print-libgcc-file-name)

diff --git a/payloads/libpayload/Makefile b/payloads/libpayload/Makefile
index bfa1abf..b319840 100644
--- a/payloads/libpayload/Makefile
+++ b/payloads/libpayload/Makefile
@@ -115,6 +115,11 @@ AR := $(AR_$(ARCH-y))

CFLAGS += $(CFLAGS_$(ARCH-y))

+ifeq ($(CC),)
+$(shell rm .xcompile)
+$(error no suitable GCC for $(ARCH-y))
+endif
+
 LIBGCC_FILE_NAME := $(shell test -r `$(CC) -print-libgcc-file-name` && \
    	      $(CC) -print-libgcc-file-name)