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

Prepend MOZTOOLS_BIN to PATH, not append #88

Merged
merged 1 commit into from Aug 9, 2016
Merged

Conversation

@vvuk
Copy link
Contributor

vvuk commented Aug 9, 2016

We need the moz tools to be at the start of the path, to not pick up random things along the way.


This change is Reviewable

@vvuk
Copy link
Contributor Author

vvuk commented Aug 9, 2016

@bors-servo r+

(no-one uses MOZTOOLS_BIN yet, hope the self-r+ is ok)

@bors-servo
Copy link
Contributor

bors-servo commented Aug 9, 2016

🔑 Insufficient privileges

@nox
Copy link
Member

nox commented Aug 9, 2016

@bors-servo
Copy link
Contributor

bors-servo commented Aug 9, 2016

📌 Commit 7ba7cfa has been approved by nox

@bors-servo
Copy link
Contributor

bors-servo commented Aug 9, 2016

Testing commit 7ba7cfa with merge 66d9eee...

bors-servo added a commit that referenced this pull request Aug 9, 2016
Prepend MOZTOOLS_BIN to PATH, not append

We need the moz tools to be at the start of the path, to not pick up random things along the way.

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

larsbergstrom commented Aug 9, 2016

@larsbergstrom larsbergstrom merged commit 0f2ea34 into servo:master Aug 9, 2016
1 of 2 checks passed
1 of 2 checks passed
homu Testing commit 7ba7cfa with merge 66d9eee...
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
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

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