Also disable dexpreopt for boot images for unbundled builds.

This fixes the tradefed build error caused by aosp/2582191.

Bug: 280440941
Test: `BUILD_BROKEN_DISABLE_BAZEL=true m dist` on tradefed branch.
(cherry picked from https://android-review.googlesource.com/q/commit:c74a4011e2815f2ffc3815176c958a0fdabbd621)
Merged-In: I34c9fea0a5f7110c9b2bda1986df82afa3eafc79
Change-Id: I34c9fea0a5f7110c9b2bda1986df82afa3eafc79
This commit is contained in:
Jiakai Zhang
2023-05-10 13:49:05 +01:00
committed by Cherrypicker Worker
parent e3f60ee435
commit 81cd7533c9
2 changed files with 12 additions and 3 deletions

View File

@@ -545,8 +545,10 @@ endif
TARGET_BUILD_USE_PREBUILT_SDKS :=
DISABLE_PREOPT :=
DISABLE_PREOPT_BOOT_IMAGES :=
ifneq (,$(TARGET_BUILD_APPS)$(TARGET_BUILD_UNBUNDLED_IMAGE))
DISABLE_PREOPT := true
DISABLE_PREOPT_BOOT_IMAGES := true
endif
ifeq (true,$(TARGET_BUILD_UNBUNDLED))
ifneq (true,$(UNBUNDLED_BUILD_SDKS_FROM_SOURCE))
@@ -557,6 +559,7 @@ endif
.KATI_READONLY := \
TARGET_BUILD_USE_PREBUILT_SDKS \
DISABLE_PREOPT \
DISABLE_PREOPT_BOOT_IMAGES \
prebuilt_sdk_tools := prebuilts/sdk/tools
prebuilt_sdk_tools_bin := $(prebuilt_sdk_tools)/$(HOST_OS)/bin

View File

@@ -12,9 +12,15 @@ else ifneq (true,$(filter true,$(PRODUCT_USES_DEFAULT_ART_CONFIG)))
# would result in passing bad arguments to dex2oat and failing the build.
ENABLE_PREOPT :=
ENABLE_PREOPT_BOOT_IMAGES :=
else ifeq (true,$(DISABLE_PREOPT))
# Disable dexpreopt for libraries/apps, but do compile boot images.
ENABLE_PREOPT :=
else
ifeq (true,$(DISABLE_PREOPT))
# Disable dexpreopt for libraries/apps, but may compile boot images.
ENABLE_PREOPT :=
endif
ifeq (true,$(DISABLE_PREOPT_BOOT_IMAGES))
# Disable dexpreopt for boot images, but may compile libraries/apps.
ENABLE_PREOPT_BOOT_IMAGES :=
endif
endif
# The default value for LOCAL_DEX_PREOPT