-
Notifications
You must be signed in to change notification settings - Fork 524
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
Opening this crate in vscode on Linux leads to files being changed and an endless build loop #2777
Comments
I don't use rust-analyzer because it makes VS Code unusable with the windows-rs repo. I don't really know what to do about that. It seems more of a rust-analyzer issue to me but perhaps I'm biased. The repo is just very very large and contains not only the Adding @ChrisDenton and @riverar perhaps they have more perspective. |
A workaround might be to exclude Maybe ra would be happier if we truncated files instead of removing and recreating? |
Btw, I don't think doing codegen in build scripts is too bizarre. While it's rarer then other codegen methods (e.g. macros) it's not exactly unheard of. |
Yeah codegen in build.rs is not uncommon and I think RA usually handles it fine. Not sure why it is causing a problem with this crate. It does seem like build.rs is changing files in the |
That's so I can use git diff during github builds to verify nothing has changed. |
Summary
To reproduce, download this crate and open it in vscode on Linux with rust-analyzer installed. (I did that since I needed to grep for some things the crate does to research Windows-related issues in my code, and I don't have a Windows machine.) This leads to changes showing up in
git diff
, and rust-analyzer hangs in an endless build loop. I reported this with RA at rust-lang/rust-analyzer#16250 but they are saying there isn't much they can do; the reason is that this crate modifies its own source in a build script which causes a reload which causes a build which modifies the source again ad nauseam.Crate manifest
No response
Crate code
No response
The text was updated successfully, but these errors were encountered: