diff options
author | Sasha Smundak <asmundak@google.com> | 2021-08-02 19:36:31 -0700 |
---|---|---|
committer | Cole Faust <colefaust@google.com> | 2021-12-03 10:55:39 -0800 |
commit | bd8965a75d306a510959e991d8c28b329c11ef01 (patch) | |
tree | e2dbe6a4ffb5f22747eafc7d40def6a7b29c7dcc | |
parent | f3623d2d23840ea44ca4704ea2965f93c78f9f56 (diff) | |
download | bramble-bd8965a75d306a510959e991d8c28b329c11ef01.tar.gz |
Replace PHONE_CAR_BOARD_CONFIG with more specific PHONE_CAR_BOARD_PRODUCT
The statement
```
include $(PHONE_CAR_BOARD_CONFIG)
```
is difficult to translate to the Starlark configuration file. Fortunately,
all the potential values of this variable have have similar structure, so
```
PHONE_CAR_BOARD_CONFIG := device/google_car/foo/BoardConfig.mk
....
include $(PHONE_CARD_BOARD_CONFIG)
```
can be changed to
```
PHONE_CAR_BOARD_PRODUCT := foo
....
include device/google_car/$(PHONE_CAR_BOARD_PRODUCT)/BoardConfig.mk
```
Bug: 190051051
Test: treehugger
Change-Id: If1f4a9733b89157ed8153e4ae930563ec3ca422e
-rw-r--r-- | bramble/BoardConfig.mk | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/bramble/BoardConfig.mk b/bramble/BoardConfig.mk index 5c402d0..ebd4a73 100644 --- a/bramble/BoardConfig.mk +++ b/bramble/BoardConfig.mk @@ -13,8 +13,8 @@ # See the License for the specific language governing permissions and # limitations under the License. # -ifdef PHONE_CAR_BOARD_CONFIG - include $(PHONE_CAR_BOARD_CONFIG) +ifdef PHONE_CAR_BOARD_PRODUCT + include device/google_car/$(PHONE_CAR_BOARD_PRODUCT)/BoardConfig.mk else TARGET_BOOTLOADER_BOARD_NAME := bramble TARGET_SCREEN_DENSITY := 420 |