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

Issue #108 - Use rexport_test_harness_main to run test with libgreen. #109

Merged
merged 1 commit into from Oct 5, 2014

Conversation

@mukilan
Copy link
Contributor

mukilan commented Oct 5, 2014

Fixes issue #108

@kmcallister
Copy link
Contributor

kmcallister commented Oct 5, 2014

I think librustuv is gone so this will break again with Servo's next Rust update. But it's still a necessary change. r+

kmcallister added a commit that referenced this pull request Oct 5, 2014
Issue #108 - Use rexport_test_harness_main to run test with libgreen.
@kmcallister kmcallister merged commit 1c5a03b into servo:master Oct 5, 2014
@jdm
Copy link
Member

jdm commented Oct 5, 2014

What happened to not touching rust-mozjs before the SM upgrade lands? :(

@kmcallister
Copy link
Contributor

kmcallister commented Oct 5, 2014

Sorry, I forgot :/

You can back out the change if you want, I think force-pushing master on this repo isn't a big deal.

mmatyas pushed a commit to mmatyas/rust-mozjs that referenced this pull request Jul 30, 2015
Fix cfg for linux/android during rust upgrade
tschneidereit pushed a commit to tschneidereit/rust-mozjs that referenced this pull request Aug 26, 2017
More portable way to set SHELL to bash.

This is the only problem I found while trying to build on NixOS (the usual problem from the unpatched virtualenv that SpiderMonkey bundles seems to have fixed itself so 🎉).

<!-- 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/109)
<!-- 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

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