Skip to content
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

Disable rustup self-updating #10970

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

madsmtm
Copy link

@madsmtm madsmtm commented Nov 14, 2024

Description

rustup has a mechanism for self-updating, which triggers whenever you request it to install a new toolchain. This is annoying to have happen in CI though, since it increases build time for no benefit (both the check itself, and the eventual update if rustup deems it necessary).

So let's disable this!

Since this is a behavioural change, it could be considered a breaking change, though I will argue that it isn't, people shouldn't need to update rustup itself during a build (rustup update to fetch a newer rustc/cargo still works fine).

Related issue:

#246, the proposed workaround in there (--no-self-update) wouldn't be necessary any more.

Check list

  • Related issue / work item is attached
  • Tests are written (if applicable)
  • Documentation is updated (if applicable)
  • Changes are tested and related VM images are successfully generated

`rustup` has a mechanism for self-updating, which triggers whenever you
request it to install a new toolchain. This is annoying to have happen in CI
though, since it increases build time for no benefit.

So let's disable this mechanism.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants