[coreboot-gerrit] New patch to review for coreboot: toolchain.inc: Improve help messages for coreboot toolchain

Martin Roth (martinroth@google.com) gerrit at coreboot.org
Fri Nov 27 06:34:58 CET 2015


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

-gerrit

commit 8aa528f6c8b1b3890fa3235141be3cd7f21f7e4d
Author: Martin Roth <martinroth at google.com>
Date:   Wed Nov 25 12:44:15 2015 -0700

    toolchain.inc: Improve help messages for coreboot toolchain
    
    Show better help text on how to compile the coreboot toolchain or use
    an unsupported toolchain.
    
    Change-Id: I64a2159d324d673784669b2464c1a2769b048678
    Signed-off-by: Martin Roth <martinroth at google.com>
---
 toolchain.inc | 10 ++++++++--
 1 file changed, 8 insertions(+), 2 deletions(-)

diff --git a/toolchain.inc b/toolchain.inc
index 0f3da83..e35b5b5 100644
--- a/toolchain.inc
+++ b/toolchain.inc
@@ -155,10 +155,16 @@ ifeq ($(CONFIG_COMPILER_GCC),y)
 ifneq ($(CONFIG_ANY_TOOLCHAIN),y)
 $(foreach arch,$(sort $(foreach stage,$(COREBOOT_STANDARD_STAGES),$(ARCH-$(stage)-y))), \
 	$(if $(shell $(CC_$(arch)) -v 2>&1 |grep -q "gcc version .*coreboot toolchain" || echo not-coreboot), \
-		$(eval COMPILERFAIL:=1)$(warning Please use the coreboot toolchain for '$(arch)' (or prove that your toolchain works))))
+		$(eval COMPILERFAIL:=1)$(warning Please use the coreboot toolchain for '$(arch)')))
 endif
 endif
 endif
 ifeq ($(COMPILERFAIL),1)
-$(error consider building our compilers: make crossgcc)
+$(warning )
+$(warning To build the entire coreboot toolchain: make crossgcc)
+$(warning For more toolchain build targets: make help_toolchain)
+$(warning )
+$(warning *** To try to use your own toolchain, run 'make menuconfig', then select the)
+$(warning *** config option: General setup -> Allow building with any toolchain)
+$(error Note that this is NOT supported.  Using it means you're on your own)
 endif



More information about the coreboot-gerrit mailing list