Skip to content

docs(user-guide): use brew install rustup instead of rustup-init#3963

Merged
rami3l merged 1 commit intomasterfrom
rami3l-patch-1
Jul 25, 2024
Merged

docs(user-guide): use brew install rustup instead of rustup-init#3963
rami3l merged 1 commit intomasterfrom
rami3l-patch-1

Conversation

@rami3l
Copy link
Copy Markdown
Member

@rami3l rami3l commented Jul 23, 2024

Homebrew/homebrew-core#177983 is using brew install rustup now, so I'm adjusting the docs accordingly:

Please note that now you need to add $(brew --prefix rustup)/bin to PATH manually to reach to the rustup-installed toolchain. Additionally, you might run brew unlink rust to make sure you won't accidentally trigger the brew-managed rust toolchain.
#1236 (comment)

The original paragraph specific to brew-installed rustup-init has been moved and extended to a more general how-to description of the two methods being used by different package managers (thus not limited to apt and brew), so our users will hopefully have even fewer chances of getting lost.

@rami3l rami3l requested a review from djc July 23, 2024 13:16
@rami3l rami3l changed the title docs(user-guide): update brew install rustup instead of rustup-init docs(user-guide): use brew install rustup instead of rustup-init Jul 23, 2024
@rami3l rami3l force-pushed the rami3l-patch-1 branch 2 times, most recently from 2133378 to 5026305 Compare July 23, 2024 13:24
@rami3l rami3l requested a review from djc July 24, 2024 13:42
@rami3l rami3l force-pushed the rami3l-patch-1 branch 2 times, most recently from c7494e9 to 4392ead Compare July 24, 2024 13:52
Copy link
Copy Markdown
Contributor

@djc djc left a comment

Choose a reason for hiding this comment

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

Looks great, thanks!

@rami3l rami3l added this pull request to the merge queue Jul 25, 2024
Merged via the queue into master with commit 99cb8e4 Jul 25, 2024
@rami3l rami3l deleted the rami3l-patch-1 branch July 25, 2024 09:53
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