-
Notifications
You must be signed in to change notification settings - Fork 13.4k
Don't download cargo twice when download-rustc is set #84750
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
Conversation
Previously, this caused a bug on NixOS: 1. bootstrap.py would download and patch stage0/cargo 2. bootstrap.py would download nightly cargo, but extract it to stage0/cargo instead of ci-rustc/cargo. 3. bootstrap.py would fail to build rustbuild because stage0/cargo wasn't patched. The "proper" fix is to extract nightly cargo to ci-rustc instead, but it doesn't seem to be necessary at all, so this just skips downloading it instead.
(rust-highfive has picked a reviewer for you, use r? to override) |
This happened because of a missing parameter to _download_component_helper, it was missing |
This is because of the rust/src/bootstrap/bootstrap.py Lines 436 to 439 in 49920bc
self.rustfmt() is the same regardless of whether stage0 is set, so it just never executes that bit of code when downloading the ci-rustc toolchain, effectively the same behavior as this PR already has.rust/src/bootstrap/bootstrap.py Lines 795 to 799 in 49920bc
|
@bors r+ |
📌 Commit 69f3ead has been approved by |
☀️ Test successful - checks-actions |
Previously, this caused a bug on NixOS:
stage0/cargo instead of ci-rustc/cargo. It would still try (and fail) to patch ci-rustc/cargo.
wasn't patched.
The "proper" fix is to extract nightly cargo to ci-rustc instead, but it
doesn't seem to be necessary at all, so this just skips downloading it
instead.
Fixes #84702