-
Notifications
You must be signed in to change notification settings - Fork 13.3k
Fix lld detection if stage0 rustc built with custom libdir #72692
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Conversation
(rust_highfive has picked a reviewer for you, use r? to override) |
The job Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
The job Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
Can we use some real targets instead of A, B, C? Maybe get list of targets with rust/src/bootstrap/builder/tests.rs Line 20 in 4512721
|
The job Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
The job Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
I've reordered test data to pass CI but I wonder if it correct. |
Reordering test data should be fine, it's testing that we are running those steps -- order doesn't really matter. However, I'm not too happy about using the first target in the |
@Mark-Simulacrum Thanks, I'll try to see if it helps. |
This looks good to me so once CI passes I'll approve, thanks! |
@Mark-Simulacrum Yes, CI passed. |
@bors r+ |
📌 Commit ce72299 has been approved by |
I'll close it in favor accepted #72728 |
Make bootstrap aware of relative libdir in stage0 compiler Follows up rust-lang#72692 Fixes rust-lang#72661
Make bootstrap aware of relative libdir in stage0 compiler Follows up rust-lang#72692 Fixes rust-lang#72661
While working on issues with stage0 built with custom libdir #72661 I found out
rust-lld
cann't be correctly detected in this case.This PR uses recently introduced #69608 to get correct place of
rust-lld
so stage0 should support it.