diff options
| author | Fredrik Roubert <roubert@google.com> | 2015-01-28 21:29:38 +0100 |
|---|---|---|
| committer | Ying Wang <wangying@android.com> | 2015-01-29 17:44:27 +0000 |
| commit | cc93f0c87cbcd7fa00e8407315c3ee60d24abd85 (patch) | |
| tree | 1727c047906cec966cbd4c87cf404a98a783ed44 | |
| parent | 67c4d916f5294e47e3da275664bb55f4518300f3 (diff) | |
| download | build-cc93f0c87cbcd7fa00e8407315c3ee60d24abd85.zip build-cc93f0c87cbcd7fa00e8407315c3ee60d24abd85.tar.gz build-cc93f0c87cbcd7fa00e8407315c3ee60d24abd85.tar.bz2 | |
Add new build flag LOCAL_DONT_DELETE_JAR_DIRS.
Normally the build function initialize-package-file will delete all
class files and all directory entries from JAR files, but sometimes
external projects (eg. ICU4J) depend on having directory entries in
their JAR files.
This change adds the flag LOCAL_DONT_DELETE_JAR_DIRS (analogous to the
flag LOCAL_DONT_DELETE_JAR_META_INF) which when set will skip deletion
of directory entries in initialize-package-file.
Change-Id: I4464b947b7528fca23925affa95e4071915f04d4
| -rw-r--r-- | core/clear_vars.mk | 1 | ||||
| -rw-r--r-- | core/definitions.mk | 4 | ||||
| -rw-r--r-- | core/host_dalvik_java_library.mk | 1 | ||||
| -rw-r--r-- | core/java_library.mk | 1 | ||||
| -rw-r--r-- | core/package_internal.mk | 1 |
5 files changed, 7 insertions, 1 deletions
diff --git a/core/clear_vars.mk b/core/clear_vars.mk index 9f339ff..db48bd6 100644 --- a/core/clear_vars.mk +++ b/core/clear_vars.mk @@ -148,6 +148,7 @@ LOCAL_LINT_FLAGS:= LOCAL_SOURCE_FILES_ALL_GENERATED:= # '',true # Don't delete the META_INF dir when merging static Java libraries. LOCAL_DONT_DELETE_JAR_META_INF:= +LOCAL_DONT_DELETE_JAR_DIRS:= LOCAL_ADDITIONAL_CERTIFICATES:= LOCAL_PREBUILT_MODULE_FILE:= LOCAL_POST_LINK_CMD:= diff --git a/core/definitions.mk b/core/definitions.mk index e276887..37f7d9d 100644 --- a/core/definitions.mk +++ b/core/definitions.mk @@ -1785,7 +1785,9 @@ endef define initialize-package-file @mkdir -p $(dir $(2)) $(hide) cp -f $(1) $(2) -$(hide) zip -qd $(2) "*.class" "*/" || true # Ignore the error when nothing to delete. +$(hide) zip -qd $(2) "*.class" \ + $(if $(strip $(PRIVATE_DONT_DELETE_JAR_DIRS)),,"*/") \ + || true # Ignore the error when nothing to delete. endef #TODO: we kinda want to build different asset packages for diff --git a/core/host_dalvik_java_library.mk b/core/host_dalvik_java_library.mk index e946b24..f8264ee 100644 --- a/core/host_dalvik_java_library.mk +++ b/core/host_dalvik_java_library.mk @@ -103,6 +103,7 @@ $(built_dex): $(full_classes_jar) $(DX) $(LOCAL_BUILT_MODULE): PRIVATE_DEX_FILE := $(built_dex) $(LOCAL_BUILT_MODULE): PRIVATE_SOURCE_ARCHIVE := $(full_classes_jarjar_jar) +$(LOCAL_BUILT_MODULE): PRIVATE_DONT_DELETE_JAR_DIRS := $(LOCAL_DONT_DELETE_JAR_DIRS) $(LOCAL_BUILT_MODULE): $(built_dex) $(java_resource_sources) @echo "Host Jar: $(PRIVATE_MODULE) ($@)" $(call initialize-package-file,$(PRIVATE_SOURCE_ARCHIVE),$@) diff --git a/core/java_library.mk b/core/java_library.mk index 61dfef0..b4e3eaa 100644 --- a/core/java_library.mk +++ b/core/java_library.mk @@ -66,6 +66,7 @@ else # !LOCAL_IS_STATIC_JAVA_LIBRARY $(common_javalib.jar): PRIVATE_DEX_FILE := $(built_dex) $(common_javalib.jar): PRIVATE_SOURCE_ARCHIVE := $(full_classes_jarjar_jar) +$(common_javalib.jar): PRIVATE_DONT_DELETE_JAR_DIRS := $(LOCAL_DONT_DELETE_JAR_DIRS) $(common_javalib.jar) : $(built_dex) $(java_resource_sources) @echo "target Jar: $(PRIVATE_MODULE) ($@)" $(call initialize-package-file,$(PRIVATE_SOURCE_ARCHIVE),$@) diff --git a/core/package_internal.mk b/core/package_internal.mk index 58db27d..49a6d9b 100644 --- a/core/package_internal.mk +++ b/core/package_internal.mk @@ -388,6 +388,7 @@ else $(LOCAL_BUILT_MODULE): PRIVATE_PRODUCT_AAPT_PREF_CONFIG := $(PRODUCT_AAPT_PREF_CONFIG) endif endif +$(LOCAL_BUILT_MODULE): PRIVATE_DONT_DELETE_JAR_DIRS := $(LOCAL_DONT_DELETE_JAR_DIRS) $(LOCAL_BUILT_MODULE): $(all_res_assets) $(jni_shared_libraries) $(full_android_manifest) @echo "target Package: $(PRIVATE_MODULE) ($@)" $(if $(PRIVATE_SOURCE_ARCHIVE),\ |