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

Expand upgrading instructions some more #95

Merged

Conversation

@fitzgen
Copy link
Member

fitzgen commented Aug 22, 2016

  • Mention building servo's bindgen fork
  • Describe running rust-mozjs/etc/bindings.sh in more detail

r? @Ms2ger


This change is Reviewable

* Mention building servo's bindgen fork
* Describe running `rust-mozjs/etc/bindings.sh` in more detail
Generate the non-`DEBUG` bindings:

```
$ cargo build

This comment has been minimized.

@fitzgen

fitzgen Aug 22, 2016

Author Member

Should this be --release as well?

This comment has been minimized.

@Ms2ger

Ms2ger Aug 23, 2016

Collaborator

No, that doesn't make a difference

@Ms2ger
Copy link
Collaborator

Ms2ger commented Aug 23, 2016

Thanks! It's great to have someone look at those instructions with fresh eyes.

@bors-servo r+

@bors-servo
Copy link
Contributor

bors-servo commented Aug 23, 2016

📌 Commit eed62fe has been approved by Ms2ger

@bors-servo
Copy link
Contributor

bors-servo commented Aug 23, 2016

Test exempted - status

@bors-servo bors-servo merged commit eed62fe into servo:master Aug 23, 2016
1 check passed
1 check passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
bors-servo added a commit that referenced this pull request Aug 23, 2016
…s2ger

Expand upgrading instructions some more

* Mention building servo's bindgen fork
* Describe running `rust-mozjs/etc/bindings.sh` in more detail

r? @Ms2ger

<!-- 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/95)
<!-- 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.