Skip to content

Fix missing dylibs #1256

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

Merged
merged 2 commits into from
Feb 14, 2018
Merged

Fix missing dylibs #1256

merged 2 commits into from
Feb 14, 2018

Conversation

fitzgen
Copy link
Member

@fitzgen fitzgen commented Feb 14, 2018

r? @emilio

Copy link
Contributor

@emilio emilio left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

r=me, with an explanation on the commit message about why is this reverted.

This reverts commit 6899c27.

The `proc_macro2` crate depends on rustc internal crates, which means that
`bindgen` would need to be run under `rustup`.

We can follow rust-lang/rust#47931 to get updates on
when this issue might be resolved and we can update `quote` again.

Fixes rust-lang#1248
@fitzgen
Copy link
Member Author

fitzgen commented Feb 14, 2018

Added to the commit message. Thanks @emilio !

@bors-servo r+

@bors-servo
Copy link

📌 Commit ad6b223 has been approved by fitzgen

@highfive highfive assigned fitzgen and unassigned emilio Feb 14, 2018
bors-servo pushed a commit that referenced this pull request Feb 14, 2018
@bors-servo
Copy link

⌛ Testing commit ad6b223 with merge 5270ef7...

@bors-servo
Copy link

💔 Test failed - status-travis

@emilio emilio merged commit 3379a90 into rust-lang:master Feb 14, 2018
@emilio
Copy link
Contributor

emilio commented Feb 14, 2018

Failures are flaky.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants