diff options
| author | Raul E Rangel <rrangel@chromium.org> | 2022-03-18 09:54:19 -0600 |
|---|---|---|
| committer | Michael Niewöhner <foss@mniewoehner.de> | 2022-03-30 07:18:06 +0000 |
| commit | 23846440dfd1a69d11647fef6a623569b87ad6f5 (patch) | |
| tree | 15117a11c7918b5245016cc158e101271b716425 /Makefile.inc | |
| parent | a84c00c9ceee83eb80ce64a8bd449491dd88d964 (diff) | |
| download | coreboot-23846440dfd1a69d11647fef6a623569b87ad6f5.tar.gz coreboot-23846440dfd1a69d11647fef6a623569b87ad6f5.tar.bz2 coreboot-23846440dfd1a69d11647fef6a623569b87ad6f5.zip | |
Makefile.inc: Explicitly delete coreboot.pre
coreboot.pre doesn't follow the standard Make conventions. It gets
modified by multiple rules, and thus we can't compute the dependencies
correctly. This means we need to manually delete it before starting the
dependency calculations.
i.e., Building firmware with the seabios payload now works correctly.
Fixes: dd6efce934f ("Makefile: Add .SECONDARY")
Signed-off-by: Raul E Rangel <rrangel@chromium.org>
Change-Id: If5fa3f0b8d314369a044658e452bd75bc7709397
Reviewed-on: https://review.coreboot.org/c/coreboot/+/62922
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Michael Niewöhner <foss@mniewoehner.de>
Diffstat (limited to 'Makefile.inc')
| -rw-r--r-- | Makefile.inc | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/Makefile.inc b/Makefile.inc index 929c236dd623..377ddff99497 100644 --- a/Makefile.inc +++ b/Makefile.inc @@ -1116,6 +1116,10 @@ ifeq ($(CONFIG_INTEL_ADD_TOP_SWAP_BOOTBLOCK),y) TS_OPTIONS := -j $(CONFIG_INTEL_TOP_SWAP_BOOTBLOCK_SIZE) endif +# coreboot.pre doesn't follow the standard Make conventions. It gets modified +# by multiple rules, and thus we can't compute the dependencies correctly. +$(shell rm -f $(obj)/coreboot.pre) + ifneq ($(CONFIG_UPDATE_IMAGE),y) $(obj)/coreboot.pre: $(objcbfs)/bootblock.bin $$(prebuilt-files) $(CBFSTOOL) $(obj)/fmap.fmap $(obj)/fmap.desc $(CBFSTOOL) $@.tmp create -M $(obj)/fmap.fmap -r $(shell cat $(obj)/fmap.desc) |