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

Make cargo test work #168

Merged
merged 1 commit into from Jul 3, 2015
Merged

Make cargo test work #168

merged 1 commit into from Jul 3, 2015

Conversation

@michaelwu
Copy link
Contributor

michaelwu commented Jul 2, 2015

Fixes #161

src/rust.rs Outdated

#[test]
pub fn dummy() {
unsafe { JS_Init(); }

This comment has been minimized.

@Ms2ger

Ms2ger Jul 2, 2015

Collaborator

Check the result, please.

This comment has been minimized.

@michaelwu

michaelwu Jul 2, 2015

Author Contributor

We don't even do that in servo.

@Ms2ger
Copy link
Collaborator

Ms2ger commented Jul 2, 2015

I don't understand the linkhack change, but then again, I don't know if anybody does. Does Servo compile with this change?

@michaelwu
Copy link
Contributor Author

michaelwu commented Jul 2, 2015

Yeah, I checked on Linux.

@michaelwu
Copy link
Contributor Author

michaelwu commented Jul 2, 2015

I mostly understand what's going on with the linkhack now. I just don't know how it worked before.

@michaelwu michaelwu force-pushed the michaelwu:fix-test branch from 376e926 to 73d2824 Jul 2, 2015
@michaelwu
Copy link
Contributor Author

michaelwu commented Jul 2, 2015

Commit updated to use assert_eq, same as in servo.

@Ms2ger
Copy link
Collaborator

Ms2ger commented Jul 3, 2015

@bors-servo
Copy link
Contributor

bors-servo commented Jul 3, 2015

📌 Commit 73d2824 has been approved by Ms2ger

@bors-servo
Copy link
Contributor

bors-servo commented Jul 3, 2015

Testing commit 73d2824 with merge 94bfd8c...

bors-servo pushed a commit that referenced this pull request Jul 3, 2015
@bors-servo
Copy link
Contributor

bors-servo commented Jul 3, 2015

☀️ Test successful - cargo-linux, cargo-mac

@bors-servo bors-servo merged commit 73d2824 into servo:master Jul 3, 2015
1 check was pending
1 check was pending
homu Testing commit 73d2824 with merge 94bfd8c...
Details
mmatyas pushed a commit to mmatyas/rust-mozjs that referenced this pull request Jul 30, 2015
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.