[coreboot-gerrit] New patch to review for coreboot: 2c251c8 build system: create proper dependency files

Patrick Georgi (pgeorgi@google.com) gerrit at coreboot.org
Fri Apr 3 11:03:19 CEST 2015


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

-gerrit

commit 2c251c8eca13aaa48ae42099dfd697aa546e5faf
Author: Patrick Georgi <pgeorgi at chromium.org>
Date:   Fri Apr 3 10:16:12 2015 +0200

    build system: create proper dependency files
    
    Tell gcc what to use as target part of the generated rule.
    
    This is inspired by the commit listed below, but rewritten to match
    upstream, and split in smaller pieces to keep intent clear.
    
    Change-Id: Ie4814143337abb3cf1e9e8db7e96201a517a17b8
    Signed-off-by: Patrick Georgi <pgeorgi at chromium.org>
    Based-On-Change-Id: I50af7dacf616e0f8ff4c43f4acc679089ad7022b
    Based-On-Signed-off-by: Julius Werner <jwerner at chromium.org>
    Based-On-Reviewed-on: https://chromium-review.googlesource.com/219170
---
 Makefile | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Makefile b/Makefile
index 03811c6..eabb959 100644
--- a/Makefile
+++ b/Makefile
@@ -232,7 +232,7 @@ ifn$(EMPTY)def $(1)-objs_$(2)_template
 de$(EMPTY)fine $(1)-objs_$(2)_template
 $(obj)/$$(1).$(1).o: src/$$(1).$(2) $(obj)/config.h $(4)
 	@printf "    CC         $$$$(subst $$$$(obj)/,,$$$$(@))\n"
-	$(CC_$(1)) $(3) -MMD $$$$(CPPFLAGS_$(1)) $$$$(CFLAGS_$(1)) -c -o $$$$@ $$$$<
+	$(CC_$(1)) -MMD $$$$(CPPFLAGS_$(1)) $$$$(CFLAGS_$(1)) -MT $$$$(@) $(3) -c -o $$$$@ $$$$<
 en$(EMPTY)def
 end$(EMPTY)if
 endef



More information about the coreboot-gerrit mailing list