-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
compile_commands.json are not getting reloaded on change. #2306
Comments
I'm able to repro the problem. Until it's fixed, using the Reload Window command would be the fastest way to get it to update. |
This is an old bug and it seriously affects my workflow. Reloading the window isn't really a good option as I often use the integrated terminal, whose state is lost on reload. Any idea on what might cause this and if it can be fixed? It would be OK to provide a manual reload command IMO. |
Related: #3864 |
Should be fixed with 0.25.0. |
Type: LanguageService
I can not find a way to reload
compile_commands.json
file other than restart vscode.Linux 4.17.6-1-ARCH
1.25.1
0.17.7
6.0.1
To Reproduce
Take example from #2305 and having everything set up, change in a line in
CMakeLists.txt
:asd
toqwe
, rebuild, resave c_cpp_properties.json. Hover overABC
inlib.cxx
in editor and it still showsasd
. Restarting vscode helps with that.Expected behavior
compile_commands.json
being reloaded on change.Additional context
I tried to debug the extension in order to investigate this issue and #2305 further and I found something interesting:
https://github.com/Microsoft/vscode-cpptools/blob/2cbc90d98b52349ad929e587c250803aee414f6f/Extension/src/LanguageServer/configurations.ts#L482
This conditional (second part
fs.existsSync(c.compileCommands)
) is not satisfied whencompileCommands
inc_cpp_properties.json
have${workspaceFolder}
in the path. I believe it should not behave like that. Setting path to absolute does not fix reloading issue though.The text was updated successfully, but these errors were encountered: