commit 1c31492ce0ad2b6bb11569997e739f2965f88b80
author Sasha Smundak <asmundak@google.com> Fri Aug 17 12:41:24 2018 -0700
committer Sasha Smundak <asmundak@google.com> Fri Aug 17 12:41:24 2018 -0700
tree 4c5cdfb6cfd299233256507f6e5e38769dc6a15c
parent 9be301eb665a9c058ad472c74f15e048ec1750d6

Do not call sort when setting ALL_DEPS.MODULES.

This particular invocation of 'sort' is expensive (it may amount to 30%
of the total ckati execution time for the clean build).

Test: verify that the result of running 'm nothing' is unchanged,
verify that the result of running 'm -j deps-license
PROJ_PATH=packages/app/% DEP_PATH=external/%' is unchanged.

Change-Id: If1cfddd4dee24559a26ecceebdf03cf49cc5a367

core/tasks/deps_licenses.mk

diff --git a/core/tasks/deps_licenses.mk b/core/tasks/deps_licenses.mk
index bb20fa0..daf986f 100644
--- a/core/tasks/deps_licenses.mk
+++ b/core/tasks/deps_licenses.mk
@@ -40,7 +40,7 @@
 endef
 
 define print-deps-license
-$(foreach m, $(ALL_DEPS.MODULES),\
+$(foreach m, $(sort $(ALL_DEPS.MODULES)),\
   $(eval m_p := $(sort $(ALL_MODULES.$(m).PATH) $(ALL_MODULES.$(m)$(TARGET_2ND_ARCH_MODULE_SUFFIX).PATH)))\
   $(if $(filter $(PROJ_PATH),$(m_p)),\
     $(eval deps :=)\