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

Use minimal profile for rustup #1550

Merged
merged 1 commit into from
Oct 15, 2019
Merged

Use minimal profile for rustup #1550

merged 1 commit into from
Oct 15, 2019

Conversation

mati865
Copy link
Contributor

@mati865 mati865 commented Oct 15, 2019

Haven't tested it but should fix issue from #1549 (comment)

@rust-highfive
Copy link

r? @gnzlbg

(rust_highfive has picked a reviewer for you, use r? to override)

@mati865
Copy link
Contributor Author

mati865 commented Oct 15, 2019

So it's good to go.
Azure image still have outdated rustup but once they upgrade it you can just uncomment lines I just added. That will make prepare step faster (less downloading and installing) and fix error if it also shows there.

@gnzlbg
Copy link
Contributor

gnzlbg commented Oct 15, 2019

@bors: r+

@bors
Copy link
Contributor

bors commented Oct 15, 2019

📌 Commit 792a9a1 has been approved by gnzlbg

@bors
Copy link
Contributor

bors commented Oct 15, 2019

⌛ Testing commit 792a9a1 with merge fab9ea1...

bors added a commit that referenced this pull request Oct 15, 2019
Use minimal profile for rustup

Haven't tested it but should fix issue from #1549 (comment)
@bors
Copy link
Contributor

bors commented Oct 15, 2019

☀️ Test successful - checks-cirrus-freebsd-10, checks-cirrus-freebsd-11, checks-cirrus-freebsd-12, status-azure
Approved by: gnzlbg
Pushing fab9ea1 to master...

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