From 2e3c3cf53776f0e354dc135a92a0f15a3b8cc22a Mon Sep 17 00:00:00 2001 From: Dan Willemsen Date: Thu, 20 Apr 2017 10:59:06 -0700 Subject: [PATCH] Fix LOCAL_MULTILIB := 64 in TARGET_TRANSLATE_2ND_ARCH builds 64-bit only apps should only be enabled if the target supports 64-bit. Test: lunch aosp_x86_arm-userdebug; m -j PMTest_Java64 Change-Id: Ibdc863730e97b1a89efb44840a1c633e9951a1d1 --- core/package.mk | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/core/package.mk b/core/package.mk index 4fe058db45..f3713fcf75 100644 --- a/core/package.mk +++ b/core/package.mk @@ -4,13 +4,15 @@ $(call record-module-type,PACKAGE) -ifeq ($(TARGET_TRANSLATE_2ND_ARCH),true) -LOCAL_MULTILIB := first -endif - my_prefix := TARGET_ include $(BUILD_SYSTEM)/multilib.mk +ifeq ($(TARGET_TRANSLATE_2ND_ARCH),true) + ifneq ($(TARGET_SUPPORTS_64_BIT_APPS)|$(my_module_multilib),|64) + my_module_multilib := first + endif +endif + ifeq ($(TARGET_SUPPORTS_32_BIT_APPS)|$(TARGET_SUPPORTS_64_BIT_APPS),true|true) # packages default to building for either architecture, # the preferred if its supported, otherwise the non-preferred.