From 45ff87c8c763205ca66cfd96b25a5ff98549ac1a Mon Sep 17 00:00:00 2001 From: Sal Savage Date: Thu, 23 Mar 2023 12:52:17 -0700 Subject: [PATCH] Use the ?= operator for assignment to avoid downstream assignment errors Using the = operator makes it so no one else can assign the system property anymore. Other build targets downstream do try to set these properties explicitly to false. Since no one sets the property before, we can use the ?= operator to apply the values, which still allows other makefiles downstream the chance to the change the value. Bug: 274950131 Test: m Change-Id: I113da6efefccef562bb13cb73087e4b5c7eca453 --- bluetooth/syna_default.mk | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/bluetooth/syna_default.mk b/bluetooth/syna_default.mk index 71ad5ab..43d9198 100644 --- a/bluetooth/syna_default.mk +++ b/bluetooth/syna_default.mk @@ -18,12 +18,12 @@ # Bluetooth LE Audio PRODUCT_PRODUCT_PROPERTIES += \ ro.bluetooth.leaudio_switcher.supported=true \ - bluetooth.profile.bap.unicast.client.enabled=true \ - bluetooth.profile.csip.set_coordinator.enabled=true \ - bluetooth.profile.hap.client.enabled=true \ - bluetooth.profile.mcp.server.enabled=true \ - bluetooth.profile.ccp.server.enabled=true \ - bluetooth.profile.vcp.controller.enabled=true \ + bluetooth.profile.bap.unicast.client.enabled?=true \ + bluetooth.profile.csip.set_coordinator.enabled?=true \ + bluetooth.profile.hap.client.enabled?=true \ + bluetooth.profile.mcp.server.enabled?=true \ + bluetooth.profile.ccp.server.enabled?=true \ + bluetooth.profile.vcp.controller.enabled?=true \ # Bluetooth HAL and Pixel extension DEVICE_MANIFEST_FILE += \