fix: repair toolchains missing their installed manifest#4954
Conversation
46a54f0 to
3a17c15
Compare
|
Ok looking |
3a17c15 to
c015a92
Compare
|
Done. I rewrote both commit messages as concise subjects and verified that the resulting tree is unchanged. |
Did you miss the earlier feedback from @rami3l? |
Yeah maybe, could you reference it? |
I meant the commit you added, but the new commit should be squashed into the first one. |
0b0f437 to
edf5664
Compare
|
Done. I folded the follow-up into the first commit, so the branch is back to two commits. |
Summary
Why
A matching update hash could skip the channel manifest download even after the installed manifest was deleted, leaving the toolchain broken while rustup reported it as unchanged.
Fixes #4825.
Validation
cargo test --features=test --test test_bonanza suite::cli_rustup::- 130 passedcargo test --features=test --lib v2_manifest_checksum_mismatch_surfaces_error- passedcargo clippy --features=test --all-targets -- -D warnings- passedcargo fmt --all -- --check- passed