-
Notifications
You must be signed in to change notification settings - Fork 2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
vscode: Add hard-coded path to compile_commands.json #20417
vscode: Add hard-coded path to compile_commands.json #20417
Conversation
When run from Windows via WSL, VS Code just won't find the `compile_commands.json` otherwise.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
I have a similar configuration on my side.
Thx :-) |
Tbh I'm not sure whether it is a good idea to include that file in the git tree. It might be automatically changed by VSCode according to the local setup and it will be necessary to double-check we don't overwrite the file by mistake :/ |
It is in There is btw. no GUI or configuration knob in VS Code to manually point it to the file. With this PR, it will just work out of the box. |
The same could apply to the |
Thx :) |
Sorry. This is not true. Once you add a file to the git tree it will be tracked. It does not matter if it in a gitignore (that also means |
I was sure that this did work for me at some point in time 😦 Looks like one has to run |
Contribution description
When run from Windows via WSL, VS Code just won't find the
compile_commands.json
otherwise.Testing procedure
make compile-commands
inexamples/hello-world
code .
to launch VS Code in Windows, connected to WSLexamples/hello-world/main.c
RIOT_BOARD
and theRIOT_CPU
macros are defined an the value is presented as"native"
when hovering with the mouseIn
master
, this doesn't work. With this PR, it should work.Issues/PRs references
None