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

Commit ff413b1c authored by Colin Cross's avatar Colin Cross Committed by Android (Google) Code Review
Browse files

Merge changes from topic "zipsync" into pi-dev 

* changes:
  Remove framework_res_package_export_deps
  Replace extract_srcjars.sh with zipsync
  Allow compiles with only srcjars
  Allow aapt2 rules to take zipped resource files
  Add LOCAL_SRCJARS support
  Remove unused .fs file suppport
  Remove unused file lists
  Remove sharding support from make
  Move aapt flags from java_common.mk to aapt_flags.mk
  Remove missed PRODUCT_AAPT_FLAGS
parents 5681de77 65c7cdf2

core/aapt2.mk

+18 −5
Original line number Diff line number Diff line
@@ -12,6 +12,7 @@ 
# - proguard_options_file

# - my_generated_res_dirs: Resources generated during the build process and we have to compile them in a single run of aapt2.

# - my_generated_res_dirs_deps: the dependency to use for my_generated_res_dirs.

# - my_generated_res_zips: Zip files containing resources

# - my_apk_split_configs: The configurations for which to generate splits.

# - built_apk_splits: The paths where AAPT should generate the splits.

#
@@ -35,7 +36,7 @@ my_overlay_resources_flat := \ 
    $(eval $(call aapt2-compile-one-resource-file-rule,$(r),$(o)))\

    $(o))



my_generated_resources_flata :=

my_resources_flata :=

# Compile generated resources

ifneq ($(my_generated_res_dirs),)

my_generated_resources_flata := $(my_compiled_res_base_dir)/gen_res.flata
@@ -44,12 +45,23 @@ $(my_generated_resources_flata) : $(my_generated_res_dirs_deps) $(AAPT2) 
	@echo "AAPT2 compile $@ <- $(PRIVATE_SOURCE_RES_DIRS)"

	$(call aapt2-compile-resource-dirs)



my_generated_resources_flata += $(my_generated_resources_flata)

my_resources_flata += $(my_generated_resources_flata)

endif



# Compile zipped resources

ifneq ($(my_generated_res_zips),)

my_zipped_resources_flata := $(my_compiled_res_base_dir)/zip_res.flata

$(my_zipped_resources_flata): PRIVATE_SOURCE_RES_ZIPS := $(my_generated_res_zips)

$(my_zipped_resources_flata) : $(my_generated_res_deps) $(AAPT2) $(ZIPSYNC)

	@echo "AAPT2 compile $@ <- $(PRIVATE_SOURCE_RES_ZIPS)"

	$(call aapt2-compile-resource-zips)



my_resources_flata += $(my_zipped_resources_flata)

endif



# Always set --pseudo-localize, it will be stripped out later for release

# builds that don't want it.

$(my_res_resources_flat) $(my_overlay_resources_flat) $(my_generated_resources_flata): \

$(my_res_resources_flat) $(my_overlay_resources_flat) $(my_resources_flata): \

  PRIVATE_AAPT2_CFLAGS := --pseudo-localize



my_static_library_resources := $(foreach l, $(call reverse-list,$(LOCAL_STATIC_ANDROID_LIBRARIES)),\
@@ -67,7 +79,7 @@ $(my_res_package): PRIVATE_AAPT_FLAGS += $(addprefix --split ,$(join $(built_apk 
endif



$(my_res_package): PRIVATE_RES_FLAT := $(my_res_resources_flat)

$(my_res_package): PRIVATE_OVERLAY_FLAT := $(my_static_library_resources) $(my_generated_resources_flata) $(my_overlay_resources_flat)

$(my_res_package): PRIVATE_OVERLAY_FLAT := $(my_static_library_resources) $(my_resources_flata) $(my_overlay_resources_flat)

$(my_res_package): PRIVATE_SHARED_ANDROID_LIBRARIES := $(my_shared_library_resources)

$(my_res_package): PRIVATE_PROGUARD_OPTIONS_FILE := $(proguard_options_file)

$(my_res_package): PRIVATE_ASSET_DIRS := $(my_asset_dirs)
@@ -95,7 +107,7 @@ endif 
$(my_res_package): $(full_android_manifest) $(my_static_library_resources) $(my_shared_library_resources)

$(my_res_package): $(my_full_asset_paths)

$(my_res_package): $(my_res_resources_flat) $(my_overlay_resources_flat) \

  $(my_generated_resources_flata) $(my_static_library_resources) \

  $(my_resources_flata) $(my_static_library_resources) \

  $(AAPT2)

	@echo "AAPT2 link $@"

	$(call aapt2-link)
@@ -118,3 +130,4 @@ my_full_asset_paths := 
my_apk_split_configs :=

my_generated_res_dirs :=

my_generated_res_dirs_deps :=

my_generated_res_zips :=

core/aapt_flags.mk

0 → 100644
+20 −0
Original line number Diff line number Diff line 
## AAPT Flags

# aapt doesn't accept multiple --extra-packages flags.

# We have to collapse them into a single --extra-packages flag here.

LOCAL_AAPT_FLAGS := $(strip $(LOCAL_AAPT_FLAGS))

ifdef LOCAL_AAPT_FLAGS

  ifeq ($(filter 0 1,$(words $(filter --extra-packages,$(LOCAL_AAPT_FLAGS)))),)

    aapt_flags := $(subst --extra-packages$(space),--extra-packages@,$(LOCAL_AAPT_FLAGS))

    aapt_flags_extra_packages := $(patsubst --extra-packages@%,%,$(filter --extra-packages@%,$(aapt_flags)))

    aapt_flags_extra_packages := $(sort $(subst :,$(space),$(aapt_flags_extra_packages)))

    LOCAL_AAPT_FLAGS := $(filter-out --extra-packages@%,$(aapt_flags)) \

        --extra-packages $(subst $(space),:,$(aapt_flags_extra_packages))

    aapt_flags_extra_packages :=

    aapt_flags :=

  endif

endif



$(LOCAL_INTERMEDIATE_TARGETS): PRIVATE_AAPT_FLAGS := $(LOCAL_AAPT_FLAGS)

$(LOCAL_INTERMEDIATE_TARGETS): PRIVATE_TARGET_AAPT_CHARACTERISTICS := $(TARGET_AAPT_CHARACTERISTICS)

$(LOCAL_INTERMEDIATE_TARGETS): PRIVATE_MANIFEST_PACKAGE_NAME := $(LOCAL_MANIFEST_PACKAGE_NAME)

$(LOCAL_INTERMEDIATE_TARGETS): PRIVATE_MANIFEST_INSTRUMENTATION_FOR := $(LOCAL_MANIFEST_INSTRUMENTATION_FOR)

core/clear_vars.mk

+0 −1
Original line number Diff line number Diff line
@@ -130,7 +130,6 @@ LOCAL_JAR_PACKAGES:= 
LOCAL_JAR_PROCESSOR:=

LOCAL_JAR_PROCESSOR_ARGS:=

LOCAL_JAVACFLAGS:=

LOCAL_JAVAC_SHARD_SIZE:=

LOCAL_JAVA_LANGUAGE_VERSION:=

LOCAL_JAVA_LAYERS_FILE:=

LOCAL_JAVA_LIBRARIES:=

core/definitions.mk

+13 −43
Original line number Diff line number Diff line
@@ -2079,6 +2079,13 @@ $(hide) $(AAPT2) compile -o $@ $(addprefix --dir ,$(PRIVATE_SOURCE_RES_DIRS)) \ 
  $(PRIVATE_AAPT2_CFLAGS) --legacy

endef



# TODO(b/74574557): use aapt2 compile --zip if it gets implemented

define aapt2-compile-resource-zips

@mkdir -p $(dir $@)

$(ZIPSYNC) -d $@.contents -l $@.list $(PRIVATE_SOURCE_RES_ZIPS)

$(hide) $(AAPT2) compile -o $@ --dir $@.tmp $(PRIVATE_AAPT2_CFLAGS) --legacy

endef



# Set up rule to compile one resource file with aapt2.

# Must be called with $(eval).

# $(1): the source file
@@ -2222,16 +2229,6 @@ $(call fetch-additional-java-source,$@.tmp) 
$(hide) tr ' ' '\n' < $@.tmp | $(NORMALIZE_PATH) | sort -u > $@

endef



# $(1): sharding number.

# $(2): Java source files paths.

define save-sharded-java-source-list

$(java_source_list_file).shard.$(1): $(2) $$(NORMALIZE_PATH)

	@echo "shard java source list: $$@"

	rm -f $$@

	$$(call dump-words-to-file,$(2),$$@.tmp)

	$(hide) tr ' ' '\n' < $$@.tmp | $$(NORMALIZE_PATH) | sort -u > $$@

endef



# Common definition to invoke javac on the host and target.

#

# $(1): javac
@@ -2241,7 +2238,9 @@ $(hide) rm -f $@ 
$(hide) rm -rf $(PRIVATE_CLASS_INTERMEDIATES_DIR) $(PRIVATE_ANNO_INTERMEDIATES_DIR)

$(hide) mkdir -p $(dir $@)

$(hide) mkdir -p $(PRIVATE_CLASS_INTERMEDIATES_DIR) $(PRIVATE_ANNO_INTERMEDIATES_DIR)

$(hide) if [ -s $(PRIVATE_JAVA_SOURCE_LIST) ] ; then \

$(if $(PRIVATE_SRCJARS),\

    $(ZIPSYNC) -d $(PRIVATE_SRCJAR_INTERMEDIATES_DIR) -l $(PRIVATE_SRCJAR_LIST_FILE) -f "*.java" $(PRIVATE_SRCJARS))

$(hide) if [ -s $(PRIVATE_JAVA_SOURCE_LIST) $(if $(PRIVATE_SRCJARS),-o -s $(PRIVATE_SRCJAR_LIST_FILE) )] ; then \

    $(SOONG_JAVAC_WRAPPER) $(JAVAC_WRAPPER) $(1) -encoding UTF-8 \

    $(if $(findstring true,$(PRIVATE_WARNINGS_ENABLE)),$(xlint_unchecked),) \

    $(if $(PRIVATE_USE_SYSTEM_MODULES), \
@@ -2260,6 +2259,7 @@ $(hide) if [ -s $(PRIVATE_JAVA_SOURCE_LIST) ] ; then \ 
    -d $(PRIVATE_CLASS_INTERMEDIATES_DIR) -s $(PRIVATE_ANNO_INTERMEDIATES_DIR) \

    $(PRIVATE_JAVACFLAGS) \

    \@$(PRIVATE_JAVA_SOURCE_LIST) \

    $(if $(PRIVATE_SRCJARS),\@$(PRIVATE_SRCJAR_LIST_FILE)) \

    || ( rm -rf $(PRIVATE_CLASS_INTERMEDIATES_DIR) ; exit 41 ) \

fi

$(if $(PRIVATE_JAVA_LAYERS_FILE), $(hide) build/make/tools/java-layers.py \
@@ -2280,45 +2280,15 @@ $(hide) $(JAR) -cf $@ $(call jar-args-sorted-files-in-directory,$(PRIVATE_CLASS_ 
$(if $(PRIVATE_EXTRA_JAR_ARGS),$(call add-java-resources-to,$@))

endef



# $(1): Javac output jar name.

# $(2): Java source list file.

# $(3): Java header libs.

# $(4): Javac sharding number.

# $(5): Javac sources deps (the arg may neeed $$ in case of containing '#')

define create-classes-full-debug.jar

$(1): PRIVATE_JAVACFLAGS := $$(LOCAL_JAVACFLAGS) $$(annotation_processor_flags)

$(1): PRIVATE_JAR_EXCLUDE_FILES := $$(LOCAL_JAR_EXCLUDE_FILES)

$(1): PRIVATE_JAR_PACKAGES := $$(LOCAL_JAR_PACKAGES)

$(1): PRIVATE_JAR_EXCLUDE_PACKAGES := $$(LOCAL_JAR_EXCLUDE_PACKAGES)

$(1): PRIVATE_DONT_DELETE_JAR_META_INF := $$(LOCAL_DONT_DELETE_JAR_META_INF)

$(1): PRIVATE_JAVA_SOURCE_LIST := $(2)

$(1): PRIVATE_ALL_JAVA_HEADER_LIBRARIES := $(3)

$(1): PRIVATE_CLASS_INTERMEDIATES_DIR := $(intermediates.COMMON)/classes$(4)

$(1): PRIVATE_ANNO_INTERMEDIATES_DIR := $(intermediates.COMMON)/anno$(4)

$(1): \

    $(2) \

    $(3) \

    $(5) \

    $$(full_java_bootclasspath_libs) \

    $$(full_java_system_modules_deps) \

    $$(layers_file) \

    $$(annotation_processor_deps) \

    $$(NORMALIZE_PATH) \

    $$(JAR_ARGS) \

    | $$(SOONG_JAVAC_WRAPPER)

	@echo "Target Java: $$@ ($$(PRIVATE_CLASS_INTERMEDIATES_DIR))"

	$$(call compile-java,$$(TARGET_JAVAC),$$(PRIVATE_ALL_JAVA_HEADER_LIBRARIES))

endef



define transform-java-to-header.jar

@echo "$($(PRIVATE_PREFIX)DISPLAY) Turbine: $(PRIVATE_MODULE)"

@mkdir -p $(dir $@)

@rm -rf $(dir $@)/classes-turbine

@mkdir $(dir $@)/classes-turbine

$(hide) if [ -s $(PRIVATE_JAVA_SOURCE_LIST) ] ; then \

$(hide) if [ -s $(PRIVATE_JAVA_SOURCE_LIST) -o -n "$(PRIVATE_SRCJARS)" ] ; then \

    $(JAVA) -jar $(TURBINE) \

    --output $@.premerged --temp_dir $(dir $@)/classes-turbine \

    --sources \@$(PRIVATE_JAVA_SOURCE_LIST) \

    --sources \@$(PRIVATE_JAVA_SOURCE_LIST) --source_jars $(PRIVATE_SRCJARS) \

    --javacopts $(PRIVATE_JAVACFLAGS) $(COMMON_JDK_FLAGS) \

    $(addprefix --bootclasspath ,$(strip \

         $(call normalize-path-list,$(PRIVATE_BOOTCLASSPATH)) \

core/droiddoc.mk

+5 −6
Original line number Diff line number Diff line
@@ -189,15 +189,15 @@ $(full_target): \ 
        $(droiddoc) \

        $(html_dir_files) \

        $(full_java_libs) \

        $(EXTRACT_SRCJARS) \

        $(ZIPSYNC) \

        $(LOCAL_SRCJARS) \

        $(LOCAL_ADDITIONAL_DEPENDENCIES)

	@echo Docs droiddoc: $(PRIVATE_OUT_DIR)

	$(hide) mkdir -p $(dir $@)

	$(hide) rm -rf $(PRIVATE_STUB_OUT_DIR) $(PRIVATE_SRCJAR_INTERMEDIATES_DIR)

	$(hide) rm -rf $(PRIVATE_STUB_OUT_DIR)

	$(call prepare-doc-source-list,$(PRIVATE_SRC_LIST_FILE),$(PRIVATE_JAVA_FILES), \

			$(PRIVATE_SOURCE_INTERMEDIATES_DIR) $(PRIVATE_ADDITIONAL_JAVA_DIR))

	$(EXTRACT_SRCJARS) $(PRIVATE_SRCJAR_INTERMEDIATES_DIR) $(PRIVATE_SRCJAR_LIST_FILE) $(PRIVATE_SRCJARS)

	$(ZIPSYNC) -d $(PRIVATE_SRCJAR_INTERMEDIATES_DIR) -l $(PRIVATE_SRCJAR_LIST_FILE) -f "*.java" $(PRIVATE_SRCJARS)

	$(hide) ( \

		$(JAVADOC) \

                -encoding UTF-8 \
@@ -243,13 +243,12 @@ else 
# For OpenJDK 8 we can use -bootclasspath to define the core libraries code.

$(full_target): PRIVATE_BOOTCLASSPATH_ARG := $(addprefix -bootclasspath ,$(PRIVATE_BOOTCLASSPATH))

endif

$(full_target): $(full_src_files) $(LOCAL_GENERATED_SOURCES) $(full_java_libs) $(EXTRACT_SRCJARS) $(LOCAL_SRCJARS) $(LOCAL_ADDITIONAL_DEPENDENCIES)

$(full_target): $(full_src_files) $(LOCAL_GENERATED_SOURCES) $(full_java_libs) $(ZIPSYNC) $(LOCAL_SRCJARS) $(LOCAL_ADDITIONAL_DEPENDENCIES)

	@echo Docs javadoc: $(PRIVATE_OUT_DIR)

	@mkdir -p $(dir $@)

	rm -rf $(PRIVATE_SRCJAR_INTERMEDIATES_DIR)

	$(call prepare-doc-source-list,$(PRIVATE_SRC_LIST_FILE),$(PRIVATE_JAVA_FILES), \

			$(PRIVATE_SOURCE_INTERMEDIATES_DIR) $(PRIVATE_ADDITIONAL_JAVA_DIR))

	$(EXTRACT_SRCJARS) $(PRIVATE_SRCJAR_INTERMEDIATES_DIR) $(PRIVATE_SRCJAR_LIST_FILE) $(PRIVATE_SRCJARS)

	$(ZIPSYNC) -d $(PRIVATE_SRCJAR_INTERMEDIATES_DIR) -l $(PRIVATE_SRCJAR_LIST_FILE) -f "*.java" $(PRIVATE_SRCJARS)

	$(hide) ( \

		$(JAVADOC) \

                -encoding UTF-8 \