Merge "Use AAPT2 for modules with manifest validation errors" am: 59fa1bcf69

am: 0cdc6edb51

Change-Id: I89da1d4e4a4ba1a9aba87f0c66f459e4ed39b82b
This commit is contained in:
Colin Cross
2018-05-21 15:34:27 -07:00
committed by android-build-merger
4 changed files with 17 additions and 12 deletions

View File

@@ -65,10 +65,12 @@ LOCAL_CERTIFICATE := platform
FrameworkCoreTests_intermediates := $(call intermediates-dir-for,APPS,$(LOCAL_PACKAGE_NAME))/test_apks/res FrameworkCoreTests_intermediates := $(call intermediates-dir-for,APPS,$(LOCAL_PACKAGE_NAME))/test_apks/res
LOCAL_RESOURCE_DIR := $(FrameworkCoreTests_intermediates) $(LOCAL_PATH)/res LOCAL_RESOURCE_DIR := $(FrameworkCoreTests_intermediates) $(LOCAL_PATH)/res
# Disable AAPT2 to fix: # Disable AAPT2 because the hacks below depend on the AAPT rules implementation
# frameworks/base/core/tests/coretests/AndroidManifest.xml:26: error: unknown element <meta-data> found.
# TODO(b/79755007): Re-enable AAPT2 when it supports the missing features.
LOCAL_USE_AAPT2 := false LOCAL_USE_AAPT2 := false
# When AAPT2 is enabled it will need --warn-manifest-validation to fix:
# frameworks/base/core/tests/coretests/AndroidManifest.xml:26: error: unknown element <meta-data> found.
# TODO(b/79755007): Remove when AAPT2 recognizes the manifest elements.
# LOCAL_AAPT_FLAGS += --warn-manifest-validation
include $(BUILD_PACKAGE) include $(BUILD_PACKAGE)
# Rules to copy all the test apks to the intermediate raw resource directory # Rules to copy all the test apks to the intermediate raw resource directory

View File

@@ -7,10 +7,11 @@ LOCAL_SRC_FILES := $(call all-subdir-java-files)
LOCAL_PACKAGE_NAME := install_multi_package LOCAL_PACKAGE_NAME := install_multi_package
# Disable AAPT2 to fix: LOCAL_USE_AAPT2 := true
# Disable AAPT2 manifest checks to fix:
# frameworks/base/core/tests/coretests/apks/install_multi_package/AndroidManifest.xml:46: error: unexpected element <package> found in <manifest>. # frameworks/base/core/tests/coretests/apks/install_multi_package/AndroidManifest.xml:46: error: unexpected element <package> found in <manifest>.
# TODO(b/79755007): Re-enable AAPT2 when it supports the missing features. # TODO(b/79755007): Remove when AAPT2 recognizes the manifest elements.
LOCAL_USE_AAPT2 := false LOCAL_AAPT_FLAGS += --warn-manifest-validation
include $(FrameworkCoreTests_BUILD_PACKAGE) include $(FrameworkCoreTests_BUILD_PACKAGE)
#include $(BUILD_PACKAGE) #include $(BUILD_PACKAGE)

View File

@@ -5,9 +5,10 @@ LOCAL_SRC_FILES := $(call all-subdir-java-files)
LOCAL_PACKAGE_NAME := install_verifier_bad LOCAL_PACKAGE_NAME := install_verifier_bad
# Disable AAPT2 to fix: LOCAL_USE_AAPT2 := true
# Disable AAPT2 manifest checks to fix:
# frameworks/base/core/tests/coretests/apks/install_verifier_bad/AndroidManifest.xml:19: error: unexpected element <package-verifier> found in <manifest>. # frameworks/base/core/tests/coretests/apks/install_verifier_bad/AndroidManifest.xml:19: error: unexpected element <package-verifier> found in <manifest>.
# TODO(b/79755007): Re-enable AAPT2 when it supports the missing features. # TODO(b/79755007): Remove when AAPT2 recognizes the manifest elements.
LOCAL_USE_AAPT2 := false LOCAL_AAPT_FLAGS += --warn-manifest-validation
include $(FrameworkCoreTests_BUILD_PACKAGE) include $(FrameworkCoreTests_BUILD_PACKAGE)

View File

@@ -5,9 +5,10 @@ LOCAL_SRC_FILES := $(call all-subdir-java-files)
LOCAL_PACKAGE_NAME := install_verifier_good LOCAL_PACKAGE_NAME := install_verifier_good
# Disable AAPT2 to fix: LOCAL_USE_AAPT2 := true
# Disable AAPT2 manifest checks to fix:
# frameworks/base/core/tests/coretests/apks/install_verifier_good/AndroidManifest.xml:19: error: unexpected element <package-verifier> found in <manifest>. # frameworks/base/core/tests/coretests/apks/install_verifier_good/AndroidManifest.xml:19: error: unexpected element <package-verifier> found in <manifest>.
# TODO(b/79755007): Re-enable AAPT2 when it supports the missing features. # TODO(b/79755007): Remove when AAPT2 recognizes the manifest elements.
LOCAL_USE_AAPT2 := false LOCAL_AAPT_FLAGS += --warn-manifest-validation
include $(FrameworkCoreTests_BUILD_PACKAGE) include $(FrameworkCoreTests_BUILD_PACKAGE)