Repository navigation
Zephyr fixups - #272
Zephyr fixups#272
Conversation
c9a3473 to
754f599
Compare
Use Python 3.12 with Zephyr, as later versions aren't supported - fixes #267 Fix link to docs - fixes #268 Add instructions to allow script execution for virtual environment - fixes #269 Fix argument passing to west build - fixes #270 Use current folder for Zephyr working directory, and pass -T for the zephyr workspace - fixes #271
754f599 to
ffe7133
Compare
west zephyr-export imports build_helpers, which has imported jsonschema since zephyrproject-rtos/zephyr@0f07a53, so on a fresh workspace the export died with ModuleNotFoundError - `west packages pip --install` had not run yet. Running it first matches the order in Zephyr's getting-started, and is safe: west packages only needs west and yaml, both already in the venv.
|
A couple of things seem worth tightening before merge. First, could we avoid recommending both:
and:
from an Administrator PowerShell?
If PowerShell activation is genuinely necessary, Second,
but the Linux fallback says Python 3.12 is strongly recommended and other versions may work. Could this be described as an extension compatibility pin instead? Something like:
That seems more precise and avoids making a broader claim about every Zephyr configuration. |
Zephyr docs only require the CurrentUser scope
I've removed
The Linux wording comes from the Zephyr docs, which state the 3.12 pin is only required on Windows, and is only strongly recommended otherwise.
|
Also fixup powershell instructions
Speeds up the search, now the python path isn't being overwritten in settings
Use Python 3.12 with Zephyr, as later versions aren't supported - fixes #267
Fix link to docs - fixes #268
Add instructions to allow script execution for virtual environment - fixes #269
Fix argument passing to west build - fixes #270