From 8a2b65244f3a0a6d2b1467ade0b3a5ff88cbc533 Mon Sep 17 00:00:00 2001 From: Tri Vo <trong@google.com> Date: Mon, 14 Jan 2019 17:25:17 -0800 Subject: [PATCH] Remove obsolete mapping build rules. Mapping files for previous releases are unconditionally packaged on the device. No need to account for case when BOARD_SEPOLICY_VERS and PLATFORM_SEPOLICY_VERSION are different. Bug: 119305624 Test: m selinux_policy Change-Id: I36c3c43f96870d9a71adf91c8fb8926587c5a50e --- Android.mk | 23 +++-------------------- 1 file changed, 3 insertions(+), 20 deletions(-) diff --git a/Android.mk b/Android.mk index f5f277dd4..d1bfade6b 100644 --- a/Android.mk +++ b/Android.mk @@ -556,32 +556,15 @@ LOCAL_MODULE_PATH := $(TARGET_OUT)/etc/selinux/mapping include $(BUILD_SYSTEM)/base_rules.mk -current_mapping.cil := $(intermediates)/mapping/$(PLATFORM_SEPOLICY_VERSION).cil -ifeq ($(BOARD_SEPOLICY_VERS), $(PLATFORM_SEPOLICY_VERSION)) +# TODO(b/119305624): Move product-specific sepolicy out of mapping files. # auto-generate the mapping file for current platform policy, since it needs to # track platform policy development -$(current_mapping.cil) : PRIVATE_VERS := $(PLATFORM_SEPOLICY_VERSION) -$(current_mapping.cil) : $(plat_pub_policy.cil) $(HOST_OUT_EXECUTABLES)/version_policy +$(LOCAL_BUILT_MODULE) : PRIVATE_VERS := $(PLATFORM_SEPOLICY_VERSION) +$(LOCAL_BUILT_MODULE) : $(plat_pub_policy.cil) $(HOST_OUT_EXECUTABLES)/version_policy @mkdir -p $(dir $@) $(hide) $(HOST_OUT_EXECUTABLES)/version_policy -b $< -m -n $(PRIVATE_VERS) -o $@ -else # ifeq ($(BOARD_SEPOLICY_VERS), $(PLATFORM_SEPOLICY_VERSION)) -# TODO(b/119305624): Move product-specific sepolicy out of mapping files. -prebuilt_mapping_files := $(wildcard \ - $(addsuffix /compat/$(BOARD_SEPOLICY_VERS)/$(BOARD_SEPOLICY_VERS).cil, \ - $(PLAT_PRIVATE_POLICY) $(PRODUCT_PRIVATE_POLICY))) -$(current_mapping.cil) : $(prebuilt_mapping_files) - @mkdir -p $(dir $@) - cat $^ > $@ - -prebuilt_mapping_files := -endif - -$(LOCAL_BUILT_MODULE): $(current_mapping.cil) $(ACP) - $(hide) $(ACP) $< $@ - built_mapping_cil := $(LOCAL_BUILT_MODULE) -current_mapping.cil := ################################# include $(CLEAR_VARS) -- GitLab