Make implicit 32-bit apps on 64-bit target an error

This has been a warning for a long time. Make it an error instead.

Once this has been verified to not cause any problems, it can
likely be changed to instead default to setting
TARGET_SUPPORTS_64_BIT_APPS := true.

Test: build_test
Change-Id: I0d03fed2085009c64880a594d5a3aadf72187e60
This commit is contained in:
Anton Hansson
2019-05-10 15:29:28 +01:00
parent 73c9f03b66
commit 6d25fc6657

View File

@@ -181,7 +181,7 @@ 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. \
$(error 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