From 3b81aabbd085c7c596b258b1bcc0a8ee0c0ac6ca Mon Sep 17 00:00:00 2001 From: Ying Wang Date: Mon, 5 May 2014 16:46:52 -0700 Subject: [PATCH] Add tool to package up built modules. With this change, you can package up modules while avoiding installing them to the system.img or userdata.img. - build/core/tasks/tools/package-modules.mk You can use this template to package up modules into a zip file and preserve the installed file paths. - LOCAL_PICKUP_FILES, you can use this variable to package up extra files/directories. Bug: 13585955 Change-Id: I103042b24ccf17cf5dc90c016d97ed1dd293e50b --- core/base_rules.mk | 6 ++++++ core/clear_vars.mk | 1 + core/tasks/tools/package-modules.mk | 43 +++++++++++++++++++++++++++++++++++++ 3 files changed, 50 insertions(+) create mode 100644 core/tasks/tools/package-modules.mk diff --git a/core/base_rules.mk b/core/base_rules.mk index 7a1975f05..0a67d4242 100644 --- a/core/base_rules.mk +++ b/core/base_rules.mk @@ -600,6 +600,12 @@ ALL_MODULES.$(my_register_name).BUILT := \ $(ALL_MODULES.$(my_register_name).BUILT) $(LOCAL_BUILT_MODULE) ALL_MODULES.$(my_register_name).INSTALLED := \ $(strip $(ALL_MODULES.$(my_register_name).INSTALLED) $(LOCAL_INSTALLED_MODULE)) +ifdef LOCAL_PICKUP_FILES +# Files or directories ready to pick up by the build system +# when $(LOCAL_BUILT_MODULE) is done. +ALL_MODULES.$(my_register_name).PICKUP_FILES := \ + $(ALL_MODULES.$(my_register_name).PICKUP_FILES) $(LOCAL_PICKUP_FILES) +endif ALL_MODULES.$(my_register_name).REQUIRED := \ $(strip $(ALL_MODULES.$(my_register_name).REQUIRED) $(LOCAL_REQUIRED_MODULES) \ $(LOCAL_REQUIRED_MODULES_$(TARGET_$(LOCAL_2ND_ARCH_VAR_PREFIX)ARCH))) diff --git a/core/clear_vars.mk b/core/clear_vars.mk index 9d83de9ef..1902d579c 100644 --- a/core/clear_vars.mk +++ b/core/clear_vars.mk @@ -13,6 +13,7 @@ LOCAL_BUILT_MODULE_STEM:= OVERRIDE_BUILT_MODULE_PATH:= LOCAL_INSTALLED_MODULE:= LOCAL_INSTALLED_MODULE_STEM:= +LOCAL_PICKUP_FILES:= LOCAL_UNINSTALLABLE_MODULE:= LOCAL_INTERMEDIATE_TARGETS:= LOCAL_UNSTRIPPED_PATH:= diff --git a/core/tasks/tools/package-modules.mk b/core/tasks/tools/package-modules.mk new file mode 100644 index 000000000..25a4e3fcb --- /dev/null +++ b/core/tasks/tools/package-modules.mk @@ -0,0 +1,43 @@ +# Package up modules to a zip file. +# It preserves the install path of the modules' installed files. +# +# Input variables: +# my_modules: a list of module names +# my_package_name: the name of the output zip file. +# Output variables: +# my_package_zip: the path to the output zip file. +# +# + +my_staging_dir := $(call intermediates-dir-for,PACKAGING,$(my_package_name)) +my_built_modules := +my_copy_pairs := +my_pickup_files := + +# Search for modules' built files and installed files; +# Calculate the dest files in the output zip file. +$(foreach m,$(my_modules),\ + $(if $(ALL_MODULES.$(m).INSTALLED),,\ + $(warning Unknown installed file for module '$(m)'))\ + $(eval my_pickup_files += $(ALL_MODULES.$(m).PICKUP_FILES))\ + $(foreach i,$(filter $(TARGET_OUT_ROOT)/%,$(ALL_MODULES.$(m).INSTALLED)),\ + $(eval b := $(filter %$(suffix $(i)),$(filter $(TARGET_OUT_ROOT)/%,$(ALL_MODULES.$(m).BUILT))))\ + $(if $(filter 1,$(words $(b))),\ + $(eval my_built_modules += $(b))\ + $(eval my_copy_pairs += $(b):$(patsubst $(PRODUCT_OUT)/%,$(my_staging_dir)/%,$(i))),\ + $(warning Unexpected module built file '$(b)' for module '$(m)'))\ + )) + +my_package_zip := $(my_staging_dir)/$(my_package_name).zip +$(my_package_zip): PRIVATE_COPY_PAIRS := $(my_copy_pairs) +$(my_package_zip): PRIVATE_PICKUP_FILES := $(my_pickup_files) +$(my_package_zip) : $(my_built_modules) + @echo "Package $@" + @rm -rf $(dir $@) && mkdir -p $(dir $@) + $(hide) $(foreach p, $(PRIVATE_COPY_PAIRS), \ + $(eval pair := $(subst :,$(space),$(p)))\ + mkdir -p $(dir $(word 2,$(pair))); \ + cp -rf $(word 1,$(pair)) $(word 2,$(pair));) + $(hide) $(foreach f, $(PRIVATE_PICKUP_FILES), \ + cp -rf $(f) $(dir $@);) + $(hide) cd $(dir $@) && zip -rq $(notdir $@) * -- 2.11.0