Skip to content
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

renaming library/std items gets InvalidParams: Cannot rename non-local definition #17633

Closed
workingjubilee opened this issue Jul 19, 2024 · 1 comment
Labels
A-project-model project model and workspace related issues C-bug Category: bug

Comments

@workingjubilee
Copy link
Member

Not all types inside libstd have set-in-stone names that cannot be changed: some are in fact quite arbitrary and deeply internal. It seems that because r-a precomputes CrateOrigin::Lang for std, rename-refactors will never work in libstd. This is because of #15232 I presume.

The "problem" is just that this is a very funny answer when the crate that you have a file open from is uhhh, rust/library/std and the file is library/std/src/sys/pal/unix/stack_overflow.rs and such.

I will go back to multicursor editing, I suppose.

@workingjubilee workingjubilee added the C-bug Category: bug label Jul 19, 2024
@lnicola lnicola added the A-project-model project model and workspace related issues label Jul 19, 2024
@Veykril
Copy link
Member

Veykril commented Jul 19, 2024

Yes that is known (that the standard library in the rustc repo is not well supported in r-a) and we aren't going to fix that from our side sorry. I've spent a lot of time trying to make the rustc workspace work nicely with r-a but I've given up, its just not a normal cargo workspace as much as bootstrap is trying to make it look like one. This will have to wait on rust-lang/rust#120611 which hopefully fixes this by telling r-a the concrete structure of the workspace that bootstrap actually knows about (instead of us having to somehow emulate the same).

@Veykril Veykril closed this as completed Jul 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-project-model project model and workspace related issues C-bug Category: bug
Projects
None yet
Development

No branches or pull requests

3 participants