From 6d25fc665719dbafed1bc292ee01f4039a08b9d7 Mon Sep 17 00:00:00 2001 From: Anton Hansson Date: Fri, 10 May 2019 15:29:28 +0100 Subject: [PATCH] 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 --- core/board_config.mk | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/core/board_config.mk b/core/board_config.mk index 2580a33fa9..7e55bfca20 100644 --- a/core/board_config.mk +++ b/core/board_config.mk @@ -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