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

Difference between static.rust-lang.org/rustup.sh and sh.rustup.rs #1507

Closed
konstin opened this issue Sep 16, 2018 · 5 comments · Fixed by #2124
Closed

Difference between static.rust-lang.org/rustup.sh and sh.rustup.rs #1507

konstin opened this issue Sep 16, 2018 · 5 comments · Fixed by #2124

Comments

@konstin
Copy link

konstin commented Sep 16, 2018

Currently, there are two rustup scripts used in the wild: https://static.rust-lang.org/rustup.sh and https://sh.rustup.rs. Both install rustup, but have different command line options. It would be great if there was some explanation on what the first script is and why one should use the first or second one.

@kinnison
Copy link
Contributor

@rust-lang/release Does this remain the case, or is the rustup-init.sh copied over during the release to be present at the top level too?

@pietroalbini
Copy link
Member

I don't think it's still copied. We can start doing that again without any issue though.

@pietroalbini
Copy link
Member

The file has been updated. @kinnison will work on adding the update command to our release process.

@konstin
Copy link
Author

konstin commented Nov 12, 2019

So the resolution is that those two scripts should be identical, currently aren't, but will be in the future? Is there any documentation about this?

@kinnison
Copy link
Contributor

@konstin The /rustup.sh script was the old shell implementation of Rustup, it's now the same as rustup-init.sh -- with #2124 we've ensured the next release will also keep them in-sync. We're likely to start using https://static.rust-lang.org/rustup.sh as the canonical installation URL at some point in the near future too. At that point, that URL will be properly documented once more :D.

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 a pull request may close this issue.

3 participants