Skip to contentSkip to conversation

Add back rootfs-* dependencies to "source" and "legal-info" targets

PATCH New Fabio Porcedda 2 participants 2 messages Mar 13, 2014 – Mar 29, 2014 Kbuild

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 Add back rootfs-* dependencies to "source" and "legal-info" targets

After adding support top-level parallel make the rootfs-* dependencies were not anymore considered for the "source" and "legal-info" targets because the rootfs-* targets were removed from TARGETS variable and placed in the TARGETS_ROOTFS variable so to fix the issue use use both "TARGETS" and "TARGETS_ROOTFS" variables.

Cc Vincent Stehl? <[email protected]>
Cc Thomas De Schampheleire <[email protected]>
---
 Makefile | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)
Makefile +2 -1
index 481f779..f7b7d84 100644
@@ -418,7 +418,8 @@ TARGETS_DIRCLEAN := $(patsubst %,%-dirclean,$(TARGETS))
 # variable for each enabled target.
 # Notice: this only works for newstyle gentargets/autotargets packages
 TARGETS_HOST_DEPS = $(sort $(filter host-%,$(foreach dep,\
-		$(addsuffix _DEPENDENCIES,$(call UPPERCASE,$(TARGETS))),\
+		$(addsuffix _DEPENDENCIES,\
+			$(call UPPERCASE,$(TARGETS) $(TARGETS_ROOTFS))),\
 		$($(dep)))))
 # Host packages can in turn have their own dependencies. Likewise find
 # all the package names listed in the HOST_<PKG>_DEPENDENCIES for each
Signature · 1 line
1.8.3.2
#1 lore raw

PATCH 1/1 Add back rootfs-* dependencies to "source" and "legal-info" targets

Dear Fabio Porcedda,

Quoted 6 lines · Fabio Porcedda

After adding support top-level parallel make the rootfs-* dependencies were not anymore considered for the "source" and "legal-info" targets because the rootfs-* targets were removed from TARGETS variable and placed in the TARGETS_ROOTFS variable so to fix the issue use use both "TARGETS" and "TARGETS_ROOTFS" variables.

Quoted 6 lines
Cc Vincent Stehl? <[email protected]>
Cc Thomas De Schampheleire <[email protected]>
---
 Makefile | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

Applied, thanks.

Thomas

Signature · 3 lines
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com