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

Commit 1fb2b8b3 authored by Anton Hansson's avatar Anton Hansson Committed by android-build-merger
Browse files

Merge "Warn when defining a 32-bit product on a 64-bit device." am: 4788f5b0

am: bfc78def

Change-Id: I4de63c3b9628c44c4f7f96c082db1a6ac1d4d130
parents a33263ab bfc78def
Loading
Loading
Loading
Loading
+9 −1
Original line number Diff line number Diff line
@@ -374,10 +374,18 @@ TARGET_PREFER_32_BIT_APPS := true
TARGET_PREFER_32_BIT_EXECUTABLES := true
endif

ifeq (,$(TARGET_SUPPORTS_32_BIT_APPS)$(TARGET_SUPPORTS_64_BIT_APPS))
ifeq (,$(filter true,$(TARGET_SUPPORTS_32_BIT_APPS) $(TARGET_SUPPORTS_64_BIT_APPS)))
  TARGET_SUPPORTS_32_BIT_APPS := true
endif

# Sanity check to warn about likely cryptic errors later in the build.
ifeq ($(TARGET_IS_64_BIT),true)
  ifeq (,$(filter true false,$(TARGET_SUPPORTS_64_BIT_APPS)))
    $(warning Building a 32-bit-app-only product on a 64-bit device. \
      If this is intentional, set TARGET_SUPPORTS_64_BIT_APPS := false)
  endif
endif

# "ro.product.cpu.abilist32" and "ro.product.cpu.abilist64" are
# comma separated lists of the 32 and 64 bit ABIs (in order of
# preference) that the target supports. If TARGET_CPU_ABI_LIST_{32,64}_BIT
+3 −1
Original line number Diff line number Diff line
@@ -8,7 +8,9 @@ my_prefix := TARGET_
include $(BUILD_SYSTEM)/multilib.mk

ifeq ($(TARGET_TRANSLATE_2ND_ARCH),true)
  ifneq ($(TARGET_SUPPORTS_64_BIT_APPS)|$(my_module_multilib),|64)
  ifeq ($(TARGET_SUPPORTS_64_BIT_APPS),true)
    my_module_multilib := first
  else ifneq ($(my_module_multilib),64)
    my_module_multilib := first
  endif
endif