Skip to content

Allow using inherited RUSTUP_UPDATE_ROOT variable in rustup-init.sh.#1495

Merged
nrc merged 1 commit intorust-lang:masterfrom
crlf0710:master
Jan 7, 2019
Merged

Allow using inherited RUSTUP_UPDATE_ROOT variable in rustup-init.sh.#1495
nrc merged 1 commit intorust-lang:masterfrom
crlf0710:master

Conversation

@crlf0710
Copy link
Copy Markdown
Member

@crlf0710 crlf0710 commented Sep 1, 2018

This also makes the usage of the variable consistent with rust version by moving the "/dist" part to usage site.

For people in mainland China, Amazon web hosting and cdn services are inaccessible. Using this patch, we can set the RUSTUP_UPDATE_ROOT variable beforehand to ask rustup-init.sh to download rustup-init.exe from a mirror site.

Also makes the usage of the variable consistent with rust version by moving the "/dist" part to usage site.
@lucasjinreal
Copy link
Copy Markdown

Will this merge? this is really useful for those who can not access to official website

@kinnison
Copy link
Copy Markdown
Contributor

kinnison commented Jan 6, 2019

This change seems harmless and, given the comments about China, useful. I think it'd be okay to merge.

Copy link
Copy Markdown
Member

@nrc nrc left a comment

Choose a reason for hiding this comment

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

Thanks! Sorry it took a while to merge.

@nrc nrc merged commit de12a26 into rust-lang:master Jan 7, 2019
AJ-Ianozi pushed a commit to AJ-Ianozi/getada-download that referenced this pull request Oct 12, 2023
Allow using inherited RUSTUP_UPDATE_ROOT variable in rustup-init.sh.
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.

4 participants