[coreboot-gerrit] New patch to review for coreboot: futility: don't pass CFLAGS into futility's build

Patrick Georgi (pgeorgi@google.com) gerrit at coreboot.org
Mon Apr 4 11:49:34 CEST 2016


Patrick Georgi (pgeorgi at google.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/14237

-gerrit

commit e46a1b72ca88007b3dd24f309b618633774b3d02
Author: Patrick Georgi <pgeorgi at google.com>
Date:   Wed Mar 30 20:59:29 2016 +0200

    futility: don't pass CFLAGS into futility's build
    
    cros_sdk puts weird stuff into CFLAGS and we never cares because we
    don't use CFLAGS. futility's Makefiles do.
    
    BUG=chromium:595715
    BRANCH=none
    TEST=none
    
    Change-Id: I512d5adb55cad8b31dc29d9c076ecd5d9c701cf6
    Signed-off-by: Patrick Georgi <pgeorgi at chromium.org>
    Original-Commit-Id: 58739332ddba7ef759aac37f3a4410dd487f210f
    Original-Change-Id: I66898c7e66d808047b0326c7471c64eaae950b15
    Original-Signed-off-by: Patrick Georgi <pgeorgi at google.com>
    Original-Reviewed-on: https://chromium-review.googlesource.com/336436
    Original-Commit-Ready: Patrick Georgi <pgeorgi at chromium.org>
    Original-Tested-by: Patrick Georgi <pgeorgi at chromium.org>
    Original-Reviewed-by: Patrick Georgi <pgeorgi at chromium.org>
---
 util/futility/Makefile.inc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/util/futility/Makefile.inc b/util/futility/Makefile.inc
index 26b218d..d37be4b 100644
--- a/util/futility/Makefile.inc
+++ b/util/futility/Makefile.inc
@@ -2,7 +2,7 @@ additional-dirs += $(objutil)/futility
 
 $(objutil)/futility/build/futility/futility:
 	@printf "    MAKE       $(subst $(objutil)/,,$(@))\n"
-	$(MAKE) -C $(VB_SOURCE) \
+	unset CFLAGS LDFLAGS; $(MAKE) -C $(VB_SOURCE) \
 		BUILD=$(abspath $@/../..) \
 		CC="$(HOSTCC)" \
 		V=$(V) \



More information about the coreboot-gerrit mailing list