[coreboot-gerrit] Patch set updated for coreboot: toolchain.inc: Skip how to use any toolchain if it's selected

Martin Roth (martinroth@google.com) gerrit at coreboot.org
Wed Dec 30 23:55:11 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/12679

-gerrit

commit 31ef2d1449f7bc3601c4ab015372e8e216cf40a4
Author: Martin Roth <martinroth at google.com>
Date:   Mon Dec 7 14:24:57 2015 -0700

    toolchain.inc: Skip how to use any toolchain if it's selected
    
    If ANY_TOOLCHAIN is selected, don't bother telling the user how to
    do what they've already done.
    
    Change-Id: I7182d18a91e832aa56638ec64fe8b3b0c38cff7a
    Signed-off-by: Martin Roth <martinroth at google.com>
---
 toolchain.inc | 10 +++++++---
 1 file changed, 7 insertions(+), 3 deletions(-)

diff --git a/toolchain.inc b/toolchain.inc
index ef3ae16..7a0864e 100644
--- a/toolchain.inc
+++ b/toolchain.inc
@@ -174,9 +174,13 @@ $(warning To build just IASL: make iasl)
 endif #($(IASLFAIL),1)
 $(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)
+ifneq ($(CONFIG_ANY_TOOLCHAIN),y)
+$(warning To try to use any toolchain in your path, run 'make menuconfig', then select)
+$(warning the config option: 'General setup', and 'Allow building with any toolchain')
+$(warning Note that this is NOT supported.  Using it means you're on your own.)
+$(warning )
+endif #($(CONFIG_ANY_TOOLCHAIN),y)
+$(error Halting the build)
 endif #($(COMPILERFAIL),1)
 
 endif #($(NOCOMPILE),1)



More information about the coreboot-gerrit mailing list