-
Notifications
You must be signed in to change notification settings - Fork 468
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
copyCompileCommands to the original path makes the file empty #3214
Comments
@tocic Thanks for the bug report! We will put this on the backlog and assess when we can get to it. In the meantime, we are happy to take open source contributions! |
copyCompileCommands is equal to default db path microsoft#3214
- if copyCompileCommands is equal to default db path, we need to avoid the copy otherwise it would copy on itself
Hi, I just hit the same bug, I think the pull request just posted above fix it After some investigation, I noticed that the compilation db located in workspace/build had a very strange size (65536) |
- if copyCompileCommands is equal to default db path, we need to avoid the copy otherwise it would copy on itself
- if copyCompileCommands is equal to default db path, we need to avoid the copy otherwise it would copy on itself
I'd like to re-open this issue, as the current solution does not account for differences in path separators on Windows. Example: "cmake.buildDirectory": "${workspaceFolder}/firmware/build" on Windows gets sanitized to "cmake.copyCompileCommands": "${workspaceFolder}/firmware/build/compile_commands.json" does not get sanitized on Windows, and results in |
Brief Issue Summary
I'm using
cmake.copyCompileCommands
as a workaround to support clangd (see #654):But if the
${workspaceFolder}/build
directory is the originalcompile_commands.json
's directory then the extension makes the file empty. I expect it to either not copy the file or overwrite the file with its original contents.CMake Tools Diagnostics
Debug Log
Additional Information
No response
The text was updated successfully, but these errors were encountered: