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

Update to mozjs_sys 0.60.1 to get libc++ on macOS #21470

Merged
merged 1 commit into from Aug 22, 2018
Merged

Update to mozjs_sys 0.60.1 to get libc++ on macOS #21470

merged 1 commit into from Aug 22, 2018

Conversation

@nox
Copy link
Member

nox commented Aug 21, 2018

This change is Reviewable

@nox
Copy link
Member Author

nox commented Aug 21, 2018

@bors-servo
Copy link
Contributor

bors-servo commented Aug 21, 2018

Trying commit dcca027 with merge 3ed89e9...

bors-servo added a commit that referenced this pull request Aug 21, 2018
Update to mozjs_sys 0.60.1 to get libc++ on macOS (WIP)

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

bors-servo commented Aug 21, 2018

💔 Test failed - windows-msvc-dev

@nox
Copy link
Member Author

nox commented Aug 21, 2018

Weird issue on Windows.

@bors-servo retry

@bors-servo
Copy link
Contributor

bors-servo commented Aug 21, 2018

Trying commit dcca027 with merge f2dd113...

bors-servo added a commit that referenced this pull request Aug 21, 2018
Update to mozjs_sys 0.60.1 to get libc++ on macOS (WIP)

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

bors-servo commented Aug 21, 2018

💔 Test failed - windows-msvc-dev

@nox
Copy link
Member Author

nox commented Aug 21, 2018

Well no idea what's going on with Windows, but the macOS build succeeded. The issue is that Travis is failing on mozjs too.

https://travis-ci.org/servo/mozjs/builds/418615632

@nox nox force-pushed the macos-mozjs branch from dcca027 to 7cb0fe1 Aug 22, 2018
@nox nox changed the title Update to mozjs_sys 0.60.1 to get libc++ on macOS (WIP) Update to mozjs_sys 0.60.1 to get libc++ on macOS Aug 22, 2018
@jdm
Copy link
Member

jdm commented Aug 22, 2018

@bors-servo
Copy link
Contributor

bors-servo commented Aug 22, 2018

📌 Commit 7cb0fe1 has been approved by jdm

@highfive highfive assigned jdm and unassigned mbrubeck Aug 22, 2018
@bors-servo
Copy link
Contributor

bors-servo commented Aug 22, 2018

Testing commit 7cb0fe1 with merge 73df3ad...

bors-servo added a commit that referenced this pull request Aug 22, 2018
Update to mozjs_sys 0.60.1 to get libc++ on macOS

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

bors-servo commented Aug 22, 2018

@bors-servo bors-servo merged commit 7cb0fe1 into master Aug 22, 2018
2 of 4 checks passed
2 of 4 checks passed
Taskcluster (pull_request) TaskGroup: failure
Details
continuous-integration/appveyor/pr AppVeyor build failed
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
homu Test successful
Details
@jdm jdm mentioned this pull request Aug 23, 2018
@Ms2ger Ms2ger deleted the macos-mozjs branch Aug 24, 2018
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.