From 73b7997ba465ed345de84ee8d9e062fdf4ad5a07 Mon Sep 17 00:00:00 2001 From: Martin Roth Date: Mon, 7 Dec 2015 14:20:55 -0700 Subject: toolchain.inc: Move nocompile around entire check, Comment endifs Move the check for NOCOMPILE flag around the whole block. There's no need to test COMPILERFAIL if NOCOMPILE is set. Comment the endif lines to make it easier to understand. Signed-off-by: Martin Roth Change-Id: Id7bb5ca13e6bf1cabf4b7b2ff3256b47b966bac1 Reviewed-on: https://review.coreboot.org/12678 Tested-by: build bot (Jenkins) Reviewed-by: Patrick Georgi Reviewed-by: Paul Menzel --- toolchain.inc | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) (limited to 'toolchain.inc') diff --git a/toolchain.inc b/toolchain.inc index c03881c802..ef3ae16641 100644 --- a/toolchain.inc +++ b/toolchain.inc @@ -162,7 +162,6 @@ $(foreach arch,$(sort $(foreach stage,$(COREBOOT_STANDARD_STAGES),$(ARCH-$(stage $(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))) endif -endif ifeq ($(COMPILERFAIL),1) ifneq ($(XGCCPATH),) $(warning ) @@ -172,10 +171,12 @@ $(warning ) $(warning To build the entire coreboot toolchain: make crossgcc) ifeq ($(IASLFAIL),1) $(warning To build just IASL: make iasl) -endif +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) -endif +endif #($(COMPILERFAIL),1) + +endif #($(NOCOMPILE),1) -- cgit v1.2.3