-
-
Notifications
You must be signed in to change notification settings - Fork 15k
x test --stage 2 std uses the wrong stage when download-rustc is enabled #108768
Copy link
Copy link
Closed
Closed
Copy link
Labels
A-download-rustcArea: The `rust.download-rustc` build option.Area: The `rust.download-rustc` build option.C-bugCategory: This is a bug.Category: This is a bug.E-easyCall for participation: Easy difficulty. Experience needed to fix: Not much. Good first issue.Call for participation: Easy difficulty. Experience needed to fix: Not much. Good first issue.E-mentorCall for participation: This issue has a mentor. Use #t-compiler/help on Zulip for discussion.Call for participation: This issue has a mentor. Use #t-compiler/help on Zulip for discussion.T-bootstrapRelevant to the bootstrap subteam: Rust's build system (x.py and src/bootstrap)Relevant to the bootstrap subteam: Rust's build system (x.py and src/bootstrap)
Description
Metadata
Metadata
Assignees
Labels
A-download-rustcArea: The `rust.download-rustc` build option.Area: The `rust.download-rustc` build option.C-bugCategory: This is a bug.Category: This is a bug.E-easyCall for participation: Easy difficulty. Experience needed to fix: Not much. Good first issue.Call for participation: Easy difficulty. Experience needed to fix: Not much. Good first issue.E-mentorCall for participation: This issue has a mentor. Use #t-compiler/help on Zulip for discussion.Call for participation: This issue has a mentor. Use #t-compiler/help on Zulip for discussion.T-bootstrapRelevant to the bootstrap subteam: Rust's build system (x.py and src/bootstrap)Relevant to the bootstrap subteam: Rust's build system (x.py and src/bootstrap)
Type
Fields
Give feedbackNo fields configured for issues without a type.
cc #81930
I tried this code:
I expected to see this happen: Bootstrap uses the stage 2 compiler to run the tests.
Instead, this happened: Bootstrap uses the stage 1 compiler:
I think the right fix is to unconditionally use the stage 2 compiler in
compiler_forwhenconfig.download_rustcis set:rust/src/bootstrap/test.rs
Line 2103 in 2f16355
rust/src/bootstrap/lib.rs
Lines 1206 to 1210 in a5d2731
Meta
HEAD is 35636f9