From 7d61a8d47b100e6c16deb2b910f4014d7d9b6f28 Mon Sep 17 00:00:00 2001 From: Yifan Hong Date: Wed, 28 Mar 2018 10:27:10 -0700 Subject: [PATCH] matrix: ENFORCE_NO_UNUSED_HALS Enforce that all hals in the device manifest are in framework compatibility matrix -- at build time. Bug: 65028233 Test: build a phone without DEVICE_FRAMEWORK_COMPATIBILITY_MATRIX. Change-Id: Ibf220f80ca3621ddbf8288d372d5f40336e45248 --- compatibility_matrices/Android.mk | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/compatibility_matrices/Android.mk b/compatibility_matrices/Android.mk index dd7165d55f..23be7de2f9 100644 --- a/compatibility_matrices/Android.mk +++ b/compatibility_matrices/Android.mk @@ -127,6 +127,14 @@ endif LOCAL_ASSEMBLE_VINTF_ENV_VARS := PRODUCT_ENFORCE_VINTF_MANIFEST +# TODO(b/65028233): Enforce no "unused HALs" for devices that does not define +# DEVICE_FRAMEWORK_COMPATIBILITY_MATRIX_FILE as well +ifeq (true,$(strip $(PRODUCT_ENFORCE_VINTF_MANIFEST))) +ifdef DEVICE_FRAMEWORK_COMPATIBILITY_MATRIX_FILE +LOCAL_ASSEMBLE_VINTF_ENV_VARS_OVERRIDE := VINTF_ENFORCE_NO_UNUSED_HALS=true +endif +endif + include $(BUILD_FRAMEWORK_COMPATIBILITY_MATRIX) BUILT_SYSTEM_COMPATIBILITY_MATRIX := $(LOCAL_BUILT_MODULE)