Remove incorrect SYSROOT usage#3594
Remove incorrect SYSROOT usage#3594o01eg wants to merge 1 commit intorust-lang:masterfrom o01eg:another-fix-2874
Conversation
Temporary solution until rust-lang/rust#57117 would be fixed. Make sys_root optional to use internal rustc_driver sysroot.
|
If rust-lang/rust#57196 will be fixed it should be possible to remove rustc call. |
|
@o01eg So I'm attempting to review this. The code changes look fine to me, but I have some general questions as I've never dealt with custom sysroots so far.
Thanks for digging into this! |
As I understand sysroot is a install.prefix where all binaries, libraries, and docs are placed. Rustc searches internal libraries in /$sysroot/$libdir.
No, the best I found are commentaries in rustc sources or ask @alexcrichton. Maybe read related PRs and issues.
No. There should be system-wide installed rustc with custom libdir. When I asked about to add tests for this case in rustc I was said they don't have enough CI resources. |
|
☔ The latest upstream changes (presumably #3665) made this pull request unmergeable. Please resolve the merge conflicts. |
|
There have been some refactorings around our sysroot detection, especially when the user tries to overwrite them. Can you check if the current system does what you need? |
|
Currently I'm waiting for clippy to be fixed in rust's source tree. |
|
ping from triage @o01eg. Clippy is fixed in the current nightly. |
|
Now they broke rustdoc again. |
|
I have it fixed with patch on rust from rust-lang/rust#57101 |
|
@o01eg hasn't that PR been reverted as part of rust-lang/rust#57303? |
|
I patch it in Gentoo build script. |
Fixes #2874
Temporary solution until rust-lang/rust#57117 would be fixed.
Make
sys_rootoptional to use internal rustc_driver sysroot.