diff options
| author | Nico Huber <nico.h@gmx.de> | 2017-08-14 01:38:24 +0200 |
|---|---|---|
| committer | Nico Huber <nico.h@gmx.de> | 2017-08-15 14:22:51 +0000 |
| commit | a2ab1fd0ff6aa729bd52b71a5717fae7661ae638 (patch) | |
| tree | dcb2851dc6f9ba72ac6954e7c027fc51065e35a4 /payloads/external/Makefile.inc | |
| parent | 506bc8e2ff34565e5c50940d0ef3addb93b07faa (diff) | |
| download | coreboot-a2ab1fd0ff6aa729bd52b71a5717fae7661ae638.tar.xz | |
payloads/external/GRUB2: Sanitize Makefile
We were explicitly passing CC and TARGET_CC to configure but overwrote
that decision later by passing CC (with the value of TARGET_CC) directly
to a recursive make call. The latter overwrite was introduced because
`unexport` alone doesn't work on variables that were specified on a make
command line (they are added to MAKEOVERRIDES and passed to further re-
cursive make calls).
Instead of unexporting random variables, unexport those that were actu-
ally passed from payloads/external/Makefile.inc and clear MAKEOVERRIDES.
Do not pass OBJDUMP as that is nowhere to be found in the GRUB sources.
And, last but not least, add --disable-werror because building GRUB is
very susceptible to changes in the flex version.
Change-Id: Iaff2c72e89a5a540fe365eacb84811d5cff9d4d4
Signed-off-by: Nico Huber <nico.h@gmx.de>
Reviewed-on: https://review.coreboot.org/20990
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Arthur Heymans <arthur@aheymans.xyz>
Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net>
Reviewed-by: Martin Roth <martinroth@google.com>
Diffstat (limited to 'payloads/external/Makefile.inc')
| -rw-r--r-- | payloads/external/Makefile.inc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/payloads/external/Makefile.inc b/payloads/external/Makefile.inc index 344fe9fff6..fb50604c75 100644 --- a/payloads/external/Makefile.inc +++ b/payloads/external/Makefile.inc @@ -155,7 +155,7 @@ payloads/external/FILO/filo/build/version.h: filo grub2: $(MAKE) -C payloads/external/GRUB2 \ HOSTCC="$(HOSTCC)" \ - CC="$(CC_x86_32)" LD="$(LD_x86_32)" OBJDUMP="$(OBJDUMP_x86_32)" \ + CC="$(CC_x86_32)" LD="$(LD_x86_32)" \ OBJCOPY="$(OBJCOPY_x86_32)" STRIP="$(STRIP_x86_32)" \ CONFIG_GRUB2_MASTER=$(CONFIG_GRUB2_MASTER) \ CONFIG_GRUB2_REVISION=$(CONFIG_GRUB2_REVISION) \ |