scripts: vscode: add support for building SOF from vscode tasks #9879
Conversation
vscode tasks can execute multiple commands easily so wrap the necessary commands to use the SOF convenience script with a small script that enables the python virtual environment. Signed-off-by: Liam Girdwood <[email protected]>
lgirdwood
requested review from
plbossart,
mmaka1,
lbetlej,
dbaluta and
kv2019i
as code owners
| # Current vscode tasks have difficulty executing multiple commands. | ||
|
|
||
| # setup environment | ||
| source ~/zephyrproject/.venv/bin/activate |
There was a problem hiding this comment.
same comment as for #9878 - too many hard-coded assumptions. Let's not do this
There was a problem hiding this comment.
How about make this conditional, if test -e ~/zephyrproject/.venv/bin/activate handling the default config?
I don't really mind for vscode-task.sh -- this can make the default scase easier and non-default setups can use a symlink or something like that.
| # Current vscode tasks have difficulty executing multiple commands. | ||
|
|
||
| # setup environment | ||
| source ~/zephyrproject/.venv/bin/activate |
There was a problem hiding this comment.
How about make this conditional, if test -e ~/zephyrproject/.venv/bin/activate handling the default config?
I don't really mind for vscode-task.sh -- this can make the default scase easier and non-default setups can use a symlink or something like that.
There was a problem hiding this comment.
I have nothing against adding this script, but it seems to me its usefulness may be negligible. After all, every development environment can be different. I have two such scripts that I call from vscode as a task, one written in bash and the other in PowerShell.
Will this be an example for others?
This script allows vscode to parse the build output and ctrl-click on the warnings/errors so it is quite useful. Fwiw, if your scripts are generic then I would recommend upstreaming too. Fwiw, vscode has all the tooling for all languages/tools we use in SOF today and is cross platform (it can be our IADK UI), I think its great for increasing everyone's velocity. Its not mandatory, but if developers do use it we should make it as smooth as possible. |
vscode tasks can execute multiple commands easily so wrap the necessary commands to use the SOF convenience script with a small script that enables the python virtual environment.