Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 634f7992 authored by Ying Wang's avatar Ying Wang
Browse files

Use ALL_MODULES.$(m).INSTALLED in modules-for-tag-list 

Previously we store tag's installed modules in a separate variable
ALL_MODULE_TAGS.$(tag). However we only record the main installed file
for a module, but omit affiliated files like .odex.
With this change, we handle the tagged module the same way as modules in
PRODUCT_PACKAGES.

Change-Id: I7972528a4df5a4ba8bc98930864da9672d32d7fe
parent 57ed1ddf

core/base_rules.mk

+0 −8
Original line number Diff line number Diff line
@@ -650,14 +650,6 @@ INSTALLABLE_FILES.$(LOCAL_INSTALLED_MODULE).MODULE := $(my_register_name) 
# Keep track of all the tags we've seen.

ALL_MODULE_TAGS := $(sort $(ALL_MODULE_TAGS) $(my_module_tags))



# Add this module to the tag list of each specified tag.

# Don't use "+=". If the variable hasn't been set with ":=",

# it will default to recursive expansion.

$(foreach tag,$(my_module_tags),\

    $(eval ALL_MODULE_TAGS.$(tag) := \

        $(ALL_MODULE_TAGS.$(tag)) \

        $(LOCAL_INSTALLED_MODULE)))



# Add this module name to the tag list of each specified tag.

$(foreach tag,$(my_module_tags),\

    $(eval ALL_MODULE_NAME_TAGS.$(tag) += $(my_register_name)))

core/definitions.mk

+1 −1
Original line number Diff line number Diff line
@@ -684,7 +684,7 @@ endef 
# any of those tags.

# $(1): tag list

define modules-for-tag-list

$(sort $(foreach tag,$(1),$(ALL_MODULE_TAGS.$(tag))))

$(sort $(foreach tag,$(1),$(foreach m,$(ALL_MODULE_NAME_TAGS.$(tag)),$(ALL_MODULES.$(m).INSTALLED))))

endef



# Same as modules-for-tag-list, but operates on