[coreboot-gerrit] New patch to review for coreboot: 3c20cdf Make: Use unaltered object list for dependency inclusion

Nico Huber (nico.huber@secunet.com) gerrit at coreboot.org
Fri May 17 15:53:36 CEST 2013


Nico Huber (nico.huber at secunet.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/3258

-gerrit

commit 3c20cdf566f447e5eb15817c86725b8c16477840
Author: Nico Huber <nico.huber at secunet.com>
Date:   Wed May 15 11:47:51 2013 +0200

    Make: Use unaltered object list for dependency inclusion
    
    It looks like the inclusion of dependency files was broken for all
    ramstage objects since the list of those gets processed through the
    ramstage-postprocess macro. Fix that by taking the unaltered list
    for dependency files.
    
    See also:
    
        commit 79f9010e80a04f2e0fb0cca5759e3215dff79aff
        Author: Patrick Georgi <patrick at georgi-clan.de>
        Date:   Sun Nov 25 14:31:08 2012 +0100
    
            build system: Add hook to postprocess classes (object lists)
    
    and:
    
        commit f33e395213f0516a9256f33ede4c6bba3babb0e9
        Author: Patrick Georgi <patrick at georgi-clan.de>
        Date:   Sun Nov 25 17:10:47 2012 +0100
    
            build system: Split linking into multiple steps
    
    Change-Id: If93b1773c5d53240f98382aab11bf7f5a4649ee8
    Signed-off-by: Nico Huber <nico.huber at secunet.com>
---
 Makefile | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/Makefile b/Makefile
index 9aafe7c..dec32e5 100644
--- a/Makefile
+++ b/Makefile
@@ -242,6 +242,9 @@ $(foreach class,$(classes),$(eval $(class)-srcs:=$(sort $($(class)-srcs))))
 src-to-obj=$(addsuffix .$(1).o, $(basename $(patsubst src/%, $(obj)/%, $($(1)-srcs))))
 $(foreach class,$(classes),$(eval $(class)-objs:=$(call src-to-obj,$(class))))
 
+# Save all objs before processing them (for dependency inclusion)
+originalobjs:=$(foreach var, $(addsuffix -objs,$(classes)), $($(var)))
+
 # Call post-processors if they're defined
 $(foreach class,$(classes),\
 	$(if $(value $(class)-postprocess),$(eval $(call $(class)-postprocess,$($(class)-objs)))))
@@ -273,7 +276,7 @@ $(foreach class,$(classes), \
 foreach-src=$(foreach file,$($(1)-srcs),$(eval $(call $(1)-objs_$(subst .,,$(suffix $(file)))_template,$(subst src/,,$(basename $(file))))))
 $(eval $(foreach class,$(classes),$(call foreach-src,$(class))))
 
-DEPENDENCIES = $(allobjs:.o=.d)
+DEPENDENCIES = $(originalobjs:.o=.d)
 -include $(DEPENDENCIES)
 
 printall:



More information about the coreboot-gerrit mailing list