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-installed binaries to $PATH #762

Merged
merged 1 commit into from Dec 14, 2017
Merged

Add rustup-installed binaries to $PATH #762

merged 1 commit into from Dec 14, 2017

Conversation

@SimonSapin
Copy link
Member

SimonSapin commented Dec 13, 2017

This is a continuation of #758


This change is Reviewable

This is a continuation of #758
@SimonSapin
Copy link
Member Author

SimonSapin commented Dec 13, 2017

r? @jdm

This will hopefully fix the failure at servo/servo#19395 (comment)

@highfive highfive assigned jdm and unassigned aneeshusa Dec 13, 2017
@aneeshusa
Copy link
Member

aneeshusa commented Dec 13, 2017

@bors-servo
Copy link
Contributor

bors-servo commented Dec 13, 2017

📌 Commit 263fa24 has been approved by aneeshusa

@highfive highfive assigned aneeshusa and unassigned jdm Dec 13, 2017
@bors-servo
Copy link
Contributor

bors-servo commented Dec 13, 2017

Testing commit 263fa24 with merge a92a5fc...

bors-servo added a commit that referenced this pull request Dec 13, 2017
Add rustup-installed binaries to $PATH

This is a continuation of #758

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/saltfs/762)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Dec 13, 2017

💔 Test failed - status-travis

@aneeshusa
Copy link
Member

aneeshusa commented Dec 13, 2017

@bors-servo retry

@bors-servo
Copy link
Contributor

bors-servo commented Dec 13, 2017

Testing commit 263fa24 with merge 6baed4f...

bors-servo added a commit that referenced this pull request Dec 13, 2017
Add rustup-installed binaries to $PATH

This is a continuation of #758

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/saltfs/762)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Dec 14, 2017

☀️ Test successful - status-travis
Approved by: aneeshusa
Pushing 6baed4f to master...

@bors-servo bors-servo merged commit 263fa24 into master Dec 14, 2017
3 checks passed
3 checks passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
homu Test successful
Details
@jdm jdm removed the S-needs-deploy label Dec 14, 2017
@SimonSapin SimonSapin deleted the rustup branch Dec 20, 2017
SimonSapin added a commit that referenced this pull request Jan 8, 2018
I missed this in #762.
bors-servo added a commit that referenced this pull request Jan 8, 2018
Add rustup-installed binaries to $PATH on Mac

I missed this in #762.

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/saltfs/771)
<!-- Reviewable:end -->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

5 participants
You can’t perform that action at this time.