diff options
author | Sasha Smundak <asmundak@google.com> | 2021-08-02 19:36:31 -0700 |
---|---|---|
committer | Sasha Smundak <asmundak@google.com> | 2021-08-09 16:21:53 -0700 |
commit | 2542ccd69191c40f9ff18fd6f0293883e339c477 (patch) | |
tree | f576a0ab27116707406da401cb79a4f67e3a82f9 | |
parent | ef7a4ae222328956c58e7323c929eea34f8648b4 (diff) | |
download | bramble-2542ccd69191c40f9ff18fd6f0293883e339c477.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 |