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

Add "rustup default none" as a way to unset the default toolchain #2831

Merged
merged 1 commit into from
Sep 4, 2021

Conversation

jyn514
Copy link
Member

@jyn514 jyn514 commented Jul 21, 2021

This can be useful when you want to be extra certain you're not using the wrong
toolchain by accident.

This also changes the testsuite to make it move obvious what went wrong when
expect_err_ex fails.

@jyn514
Copy link
Member Author

jyn514 commented Jul 21, 2021

Looks spurious:

Installing Snapcraft plus dependencies
  Ensuring runner is in the lxd group...
  /usr/bin/sudo groupadd --force --system lxd
  /usr/bin/sudo usermod --append --groups lxd runner
  Installing LXD...
  /usr/bin/sudo snap refresh lxd
  error: cannot refresh "lxd": unexpectedly empty response from the server (try
         again later)
  Error: The process '/usr/bin/sudo' failed with exit code 1

Copy link
Contributor

@kinnison kinnison left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The changes look good, let's see if we can re-trigger the checks

This can be useful when you want to be extra certain you're not using the wrong
toolchain by accident.

This also changes the testsuite to make it move obvious what went wrong when
`expect_err_ex` fails.
@kinnison kinnison merged commit 38f4479 into rust-lang:master Sep 4, 2021
@jyn514 jyn514 deleted the default-none branch September 4, 2021 13:12
@jyn514
Copy link
Member Author

jyn514 commented Sep 4, 2021

Oh heh, I forgot I made this PR. Thanks for the review!

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