From 695a1268e18d882f5a0ea1003909f55aecc7080d Mon Sep 17 00:00:00 2001 From: Felix Fietkau Date: Wed, 24 Nov 2010 14:07:12 +0000 Subject: [PATCH] change the recursive dependency template to use more make evaluation - the shell command got so long that it triggered "Argument list too long" on some systems (#8231) SVN-Revision: 24131 --- include/depends.mk | 43 ++++++++++++++++++++++--------------------- 1 file changed, 22 insertions(+), 21 deletions(-) diff --git a/include/depends.mk b/include/depends.mk index 042fd4f4f2..14cdcdf7a7 100644 --- a/include/depends.mk +++ b/include/depends.mk @@ -20,28 +20,29 @@ define rdep .SILENT: $(2)_check $(2): $(2)_check + +ifneq ($(wildcard $(2)),) + $(2)_check:: + $(if $(3), \ + $(call find_md5,$(1),$(4)) > $(3).1; \ + { [ \! -f "$(3)" ] || diff $(3) $(3).1 >/dev/null; } && \ + ) \ + { \ + [ -f "$(2)_check.1" ] && mv "$(2)_check.1"; \ + $(TOPDIR)/scripts/timestamp.pl $(DEP_FINDPARAMS) $(4) -n $(2) $(1) && { \ + $(call debug_eval,$(SUBDIR),r,echo "No need to rebuild $(2)";) \ + touch -r "$(2)" "$(2)_check"; \ + } \ + } || { \ + $(call debug_eval,$(SUBDIR),r,echo "Need to rebuild $(2)";) \ + touch "$(2)_check"; \ + } + $(if $(3), mv $(3).1 $(3)) +else $(2)_check:: - if [ -f "$(2)" ]; then \ - $(if $(3), \ - $(call find_md5,$(1),$(4)) > $(3).1; \ - { [ \! -f "$(3)" ] || diff $(3) $(3).1 >/dev/null; } && \ - ) \ - { \ - [ -f "$(2)_check.1" ] && mv "$(2)_check.1"; \ - $(TOPDIR)/scripts/timestamp.pl $(DEP_FINDPARAMS) $(4) -n $(2) $(1) && { \ - $(call debug_eval,$(SUBDIR),r,echo "No need to rebuild $(2)";) \ - touch -r "$(2)" "$(2)_check"; \ - } \ - } || { \ - $(call debug_eval,$(SUBDIR),r,echo "Need to rebuild $(2)";) \ - touch "$(2)_check"; \ - }; \ - $(if $(3), mv $(3).1 $(3);) \ - else \ - $(if $(3), rm -f $(3) $(3).1;) \ - $(call debug_eval,$(SUBDIR),r,echo "Target $(2) not built";) \ - true; \ - fi + $(if $(3), rm -f $(3) $(3).1) + $(call debug_eval,$(SUBDIR),r,echo "Target $(2) not built") +endif endef -- 2.30.2