From ada36d4cff54c419479a7865278108d2ae44bc6e Mon Sep 17 00:00:00 2001 From: Martin Roth Date: Mon, 7 Dec 2015 14:27:34 -0700 Subject: toolchain.inc: Update help text, Add TODO. - Update the help text to be more informative. - Add todo about IASL - we shouldn't require it if the build doesn't use it. Change-Id: Iffeb94f78c1ae7535a8a7b9b0b9f1728301a42b3 Signed-off-by: Martin Roth Reviewed-on: https://review.coreboot.org/12680 Tested-by: build bot (Jenkins) Reviewed-by: Stefan Reinauer --- toolchain.inc | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) (limited to 'toolchain.inc') diff --git a/toolchain.inc b/toolchain.inc index 7a0864eeade6..f2a6232c77cd 100644 --- a/toolchain.inc +++ b/toolchain.inc @@ -157,10 +157,11 @@ IASLFAIL:=0 ifneq ($(CONFIG_ANY_TOOLCHAIN),y) $(foreach arch,$(sort $(foreach stage,$(COREBOOT_STANDARD_STAGES),$(ARCH-$(stage)-y))), \ $(if $(shell if [ -n "$(CC_$(arch))" ]; then $(CC_$(arch)) -v 2>&1 |grep -q "coreboot toolchain" || echo not-coreboot; else echo not-coreboot; fi), \ - $(eval COMPILERFAIL:=1)$(warning Please use the coreboot toolchain for '$(arch)'))) + $(eval COMPILERFAIL:=1)$(warning The coreboot toolchain for '$(arch)' architecture was not found.))) #if iasl doesn't match the current coreboot version, fail the test +#TODO: Figure out if iasl is even needed for the build. $(if $(shell if [ -n "$(IASL)" ]; then $(IASL) -v 2>&1 | grep -q "$(shell util/crossgcc/buildgcc -s iasl)" || echo not-coreboot; else echo not-coreboot; fi), \ - $(eval COMPILERFAIL:=1)$(eval IASLFAIL:=1)$(warning Please use the coreboot toolchain version of iasl - $(shell util/crossgcc/buildgcc -s iasl))) + $(eval COMPILERFAIL:=1)$(eval IASLFAIL:=1)$(warning The coreboot toolchain version of iasl '$(shell util/crossgcc/buildgcc -s iasl)' was not found)) endif ifeq ($(COMPILERFAIL),1) ifneq ($(XGCCPATH),) @@ -168,11 +169,11 @@ $(warning ) $(warning Path to your toolchain is currently set to '$(XGCCPATH)') endif $(warning ) -$(warning To build the entire coreboot toolchain: make crossgcc) +$(warning To build the entire coreboot toolchain: run 'make crossgcc') ifeq ($(IASLFAIL),1) -$(warning To build just IASL: make iasl) +$(warning To build just IASL: run 'make iasl') endif #($(IASLFAIL),1) -$(warning For more toolchain build targets: make help_toolchain) +$(warning For more toolchain build targets: run 'make help_toolchain') $(warning ) ifneq ($(CONFIG_ANY_TOOLCHAIN),y) $(warning To try to use any toolchain in your path, run 'make menuconfig', then select) -- cgit v1.2.3