nRF Connect for VS Code messes up environment variables

RE: nRF Connect for VS Code messes up environment variables

Håkon Alseth — Fri, 16 Sep 2022 15:05:12 GMT

Hi,

[quote user="JonasJ"]

At the end of that documentation I found this. Not sure that I understand this correctly. VS Code reads .profile but if I want to disable that feature I should set things i .profile?

Is this paragraph really correct?

[/quote]

I understand the confusion here, I'll try to explain a bit about the background of how it became this way.

Former versions of NCS/zephyr stated that you shall use environmental vars as a part of the getting started for the linux platform, similar to what is still documented for "west": https://developer.nordicsemi.com/nRF_Connect_SDK/doc/2.0.2/zephyr/develop/getting_started/index.html#get-zephyr-and-install-python-dependencies

But, it also stated that the toolchain and so forth should be placed in the local environment.

We had to pick these up with VSCode, which meant that we needed to parse the local env setup to pick them out, which per the above has been .bashrc. So, to avoid that it parses more than it should, you can move it one step up, ie. in .profile.

Kind regards,

Håkon

RE: nRF Connect for VS Code messes up environment variables

JonasJ — Thu, 15 Sep 2022 11:14:47 GMT

Thanks.

These environment variables are set in system files so it's not something I'd like to mess around in. But thanks for sharing that part of the documentation, didn't know about that variable. Can be useful.

I hope the VS Code team can figure out what is wrong. It doesn't stop med from compiling but it generates annoying noise in the terminal when build.

At the end of that documentation I found this. Not sure that I understand this correctly. VS Code reads .profile but if I want to disable that feature I should set things i .profile?

Is this paragraph really correct?

VS Code picks up environment variables set in .profile. To disable this option while still being compatible with the terminal, move the nRF Connect SDK-specific environment variables (such as ZEPHYR_TOOLCHAIN_VARIANT or GNUARMEMB_TOOLCHAIN_PATH) to .profile.

RE: nRF Connect for VS Code messes up environment variables

Håkon Alseth — Thu, 15 Sep 2022 11:07:59 GMT

Hi,

Yes, I can confirm that there can be issues with local environment on different linux distro's, as we mainly target ubuntu.

I have made the nrf VScode team aware of your issue, and it will be evaluated for future improvements.

As a workaround, can you try to check for $NRF_CONNECT_VSCODE to work around this issue?

More details can be found here in our documentation: https://nrfconnect.github.io/vscode-nrf-connect/connect/west.html#controlling-the-west-environment

Kind regards,

Håkon

RE: nRF Connect for VS Code messes up environment variables

JonasJ — Wed, 14 Sep 2022 13:46:15 GMT

Hi,

You can test this you self by sourcing the following file in a shell and launch vscode after that. You should see an error similar to my report.

foo_bar() {
    echo "This is a function in bash"
}

export -f foo_bar

I'm running Fedora 36 and it doesn't matter which project I build.

RE: nRF Connect for VS Code messes up environment variables

Håkon Alseth — Wed, 14 Sep 2022 13:35:45 GMT

Hi,

I am trying to recreate this on my end, but I'll need some added info about your setup to be able to do this.

Which distro are you using?

Can you elaborate on how and where you place this particular env variable?

Does this issue occur when compiling any example, or is it a specific one?

Kind regards,

Håkon

RE: nRF Connect for VS Code messes up environment variables

JonasJ — Wed, 14 Sep 2022 12:51:13 GMT

Hi,

VS Code correctly inherits the environment variables, no issue there. If you try to build a project using nRF Connect SDK Extension I get the errors in my post.

I assume this is because the nRF extension for VS Code reads the existing env variables but can't handle those that have newline (\n) in them and thus creates a broken env when launching the terminal to run west build.

Thanks.

RE: nRF Connect for VS Code messes up environment variables

Håkon Alseth — Wed, 14 Sep 2022 12:45:54 GMT

Hi,

Could you elaborate a bit on where these env vars are located and how you're opening vscode?

If you open vscode from the terminal where you have everything setup, it should also inherit your whole environment. Could you try this? ie. run this from terminal: code /path/to/ncs

Kind regards,

Håkon