Skip to content

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