commit a42e5f976d6d9211b75f5adb216ac02ee5dec7ca
author Yifan Hong <elsk@google.com> Tue Jul 31 13:30:38 2018 -0700
committer Yifan Hong <elsk@google.com> Wed Aug 01 18:29:44 2018 +0000
tree 4bdb1236dbe11f9d3dd90eef7dce11fd67b47bcc
parent d1c308bf406331c2c5cb488b5f8cee44a05e0ed8

Remove USE_LOGICAL_PARTITIONS override

... so that its requirements does not fallback
on the overridden USE_LOGICAL_PARTITIONS, but
PRODUCT_USE_LOGICAL_PARTITIONS.

Test: m superimage -j
Change-Id: Ib14aaf83cc8bd4336125d503d11c69a60566530e

core/Makefile

diff --git a/core/Makefile b/core/Makefile
index 03a5b79..ae8a30a 100644
--- a/core/Makefile
+++ b/core/Makefile
@@ -1157,7 +1157,7 @@
 INTERNAL_USERIMAGES_DEPS += $(MKE2FS_CONF)
 endif
 
-ifeq (true,$(USE_LOGICAL_PARTITIONS))
+ifeq (true,$(PRODUCT_USE_LOGICAL_PARTITIONS))
 
 ifeq ($(PRODUCTS.$(INTERNAL_PRODUCT).PRODUCT_SUPPORTS_VERITY),true)
   $(error vboot 1.0 doesn't support logical partition)
@@ -1166,7 +1166,7 @@
 # TODO(b/80195851): Should not define BOARD_AVB_SYSTEM_KEY_PATH without
 # BOARD_AVB_SYSTEM_DETACHED_VBMETA.
 
-endif # USE_LOGICAL_PARTITIONS
+endif # PRODUCT_USE_LOGICAL_PARTITIONS
 
 # $(1): the path of the output dictionary file
 # $(2): a subset of "system vendor cache userdata product product_services oem"

core/config.mk

diff --git a/core/config.mk b/core/config.mk
index b4ddf4b..53718d5 100644
--- a/core/config.mk
+++ b/core/config.mk
@@ -925,14 +925,14 @@
 endif
 .KATI_READONLY := PRODUCT_OTA_ENFORCE_VINTF_KERNEL_REQUIREMENTS
 
-ifeq ($(USE_LOGICAL_PARTITIONS),true)
+ifeq ($(PRODUCT_USE_LOGICAL_PARTITIONS),true)
     requirements := \
         PRODUCT_USE_DYNAMIC_PARTITION_SIZE \
         PRODUCT_BUILD_SUPER_PARTITION \
         PRODUCT_USE_FASTBOOTD \
 
     $(foreach req,$(requirements),$(if $(filter false,$($(req))),\
-        $(error USE_LOGICAL_PARTITIONS requires $(req) to be true)))
+        $(error PRODUCT_USE_LOGICAL_PARTITIONS requires $(req) to be true)))
 
     requirements :=
 

core/main.mk

diff --git a/core/main.mk b/core/main.mk
index 86b30c8..ba082ac 100644
--- a/core/main.mk
+++ b/core/main.mk
@@ -240,7 +240,7 @@
 ADDITIONAL_DEFAULT_PROPERTIES += ro.actionable_compatible_property.enabled=${PRODUCT_COMPATIBLE_PROPERTY}
 endif
 
-ADDITIONAL_PRODUCT_PROPERTIES += ro.boot.logical_partitions=$(USE_LOGICAL_PARTITIONS)
+ADDITIONAL_PRODUCT_PROPERTIES += ro.boot.logical_partitions=$(PRODUCT_USE_LOGICAL_PARTITIONS)
 
 # -----------------------------------------------------------------
 ###

core/product_config.mk

diff --git a/core/product_config.mk b/core/product_config.mk
index 93bca41..b5f2896 100644
--- a/core/product_config.mk
+++ b/core/product_config.mk
@@ -519,21 +519,18 @@
 PRODUCT_USE_LOGICAL_PARTITIONS := \
     $(strip $(PRODUCTS.$(INTERNAL_PRODUCT).PRODUCT_USE_LOGICAL_PARTITIONS))
 .KATI_READONLY := PRODUCT_USE_LOGICAL_PARTITIONS
-ifndef USE_LOGICAL_PARTITIONS
-    USE_LOGICAL_PARTITIONS := $(PRODUCT_USE_LOGICAL_PARTITIONS)
-endif
 
-# All requirements of USE_LOGICAL_PARTITIONS falls back to
-# USE_LOGICAL_PARTITIONS if not defined.
+# All requirements of PRODUCT_USE_LOGICAL_PARTITIONS falls back to
+# PRODUCT_USE_LOGICAL_PARTITIONS if not defined.
 PRODUCT_USE_DYNAMIC_PARTITION_SIZE := $(or \
     $(strip $(PRODUCTS.$(INTERNAL_PRODUCT).PRODUCT_USE_DYNAMIC_PARTITION_SIZE)),\
-    $(USE_LOGICAL_PARTITIONS))
+    $(PRODUCT_USE_LOGICAL_PARTITIONS))
 .KATI_READONLY := PRODUCT_USE_DYNAMIC_PARTITION_SIZE
 PRODUCT_BUILD_SUPER_PARTITION := $(or \
     $(strip $(PRODUCTS.$(INTERNAL_PRODUCT).PRODUCT_BUILD_SUPER_PARTITION)),\
-    $(USE_LOGICAL_PARTITIONS))
+    $(PRODUCT_USE_LOGICAL_PARTITIONS))
 .KATI_READONLY := PRODUCT_BUILD_SUPER_PARTITION
 PRODUCT_USE_FASTBOOTD := $(or \
     $(strip $(PRODUCTS.$(INTERNAL_PRODUCT).PRODUCT_USE_FASTBOOTD)),\
-    $(USE_LOGICAL_PARTITIONS))
+    $(PRODUCT_USE_LOGICAL_PARTITIONS))
 .KATI_READONLY := PRODUCT_USE_FASTBOOTD