najnesnaj / pinetime-zephyr

pinetime smartwatch nrf52 zephyr
Apache License 2.0
107 stars 20 forks source link

Add west support as external zephyr application -- tested but not working out of the box #4

Closed nordic-krch closed 4 years ago

nordic-krch commented 4 years ago

Added west.yaml to have control of dependencies (mainly zephyr). Added changes needed to make pinetime work as external (out of tree) application. Copying code into zephyr is no longer needed. Project have following structure after change: /pinetime/pinetime zephyr ...

nordic-krch commented 4 years ago

This is my first PR here. I think that long term it would be good to switch to use west and zephyr as external dependency. This way copying into zephyr is not needed and zephyr version can be maintained. Same approach is taken by nrf connect sdk (https://github.com/NordicPlayground/fw-nrfconnect-nrf) which is based on zephyr.

najnesnaj commented 4 years ago

Krzysztof,

           thank you for the contribution and advise! I'll will adapt the manual as well.

Kind Regards, Jan