lancaster-university / microbit-v2-samples

CODAL build tools and sample programs for the micro:bit v2.x
MIT License
66 stars 87 forks source link

`codal.json` `target.dev` property looks like a boolean but it isn't #63

Open microbit-carlos opened 1 year ago

microbit-carlos commented 1 year ago

The CMakeLists.txt file just check if it has any value, so if the user sets "dev": false in the json file, it will still treat it as a dev build. https://github.com/lancaster-university/microbit-v2-samples/blob/ce0d2aa3c3d0c5c656dff8f833c3d39945ed10cb/CMakeLists.txt#L67-L73

As the codal.json file in this repository contains "dev": true it's a fair expectation that changing it to false should disable it.

A fix in this repo should also be pushed upstream to lancaster-university/codal and to codal bootstrap (if applicable).