Merge "Verbose warning message for PRODUCT_INSTALL_DEBUG_POLICY_TO_SYSTEM_EXT" am: ec918431a5
am: 21013eaa18
am: bb52d2bc9d
Original change: https://android-review.googlesource.com/c/platform/build/+/1860893 Change-Id: I1525b4622f1bb5450abca544c0f0e1b9ddc24538
This commit is contained in:
@@ -445,7 +445,7 @@ _product_single_value_vars += PRODUCT_INSTALL_EXTRA_FLATTENED_APEXES
|
|||||||
|
|
||||||
# Install a copy of the debug policy to the system_ext partition, and allow
|
# Install a copy of the debug policy to the system_ext partition, and allow
|
||||||
# init-second-stage to load debug policy from system_ext.
|
# init-second-stage to load debug policy from system_ext.
|
||||||
# This option is only meant to be set by GSI products.
|
# This option is only meant to be set by compliance GSI targets.
|
||||||
_product_single_value_vars += PRODUCT_INSTALL_DEBUG_POLICY_TO_SYSTEM_EXT
|
_product_single_value_vars += PRODUCT_INSTALL_DEBUG_POLICY_TO_SYSTEM_EXT
|
||||||
|
|
||||||
# If set, metadata files for the following artifacts will be generated.
|
# If set, metadata files for the following artifacts will be generated.
|
||||||
|
@@ -420,9 +420,15 @@ ifneq ($(filter-out 0 1,$(words $(PRODUCT_ADB_KEYS))),)
|
|||||||
$(error Only one file may be in PRODUCT_ADB_KEYS: $(PRODUCT_ADB_KEYS))
|
$(error Only one file may be in PRODUCT_ADB_KEYS: $(PRODUCT_ADB_KEYS))
|
||||||
endif
|
endif
|
||||||
|
|
||||||
|
# Show a warning wall of text if non-compliance-GSI products set this option.
|
||||||
ifdef PRODUCT_INSTALL_DEBUG_POLICY_TO_SYSTEM_EXT
|
ifdef PRODUCT_INSTALL_DEBUG_POLICY_TO_SYSTEM_EXT
|
||||||
ifeq (,$(filter gsi_arm gsi_arm64 gsi_x86 gsi_x86_64,$(PRODUCT_NAME)))
|
ifeq (,$(filter gsi_arm gsi_arm64 gsi_x86 gsi_x86_64 gsi_car_arm64 gsi_car_x86_64,$(PRODUCT_NAME)))
|
||||||
$(error Only GSI products are allowed to set PRODUCT_INSTALL_DEBUG_POLICY_TO_SYSTEM_EXT)
|
$(warning PRODUCT_INSTALL_DEBUG_POLICY_TO_SYSTEM_EXT is set but \
|
||||||
|
PRODUCT_NAME ($(PRODUCT_NAME)) doesn't look like a GSI for compliance \
|
||||||
|
testing. This is a special configuration for compliance GSI, so do make \
|
||||||
|
sure you understand the security implications before setting this \
|
||||||
|
option. If you don't know what this option does, then you probably \
|
||||||
|
shouldn't set this.)
|
||||||
endif
|
endif
|
||||||
endif
|
endif
|
||||||
|
|
||||||
|
Reference in New Issue
Block a user