Skip to content

Put blocks behind a switch. #1410

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
Oct 4, 2018
Merged

Conversation

emilio
Copy link
Contributor

@emilio emilio commented Oct 4, 2018

Since #1378 broke a bunch of OSX builds.

Most people don't care about them and they're in some OSX system headers which
means that this could break normal C and C++ stuff.

This introduces --generate-block / generate_block to generate these signatures,
and adds tests so that this is getting tested.

emilio added 2 commits October 4, 2018 16:15
Since rust-lang#1378 broke a bunch of OSX builds.

Most people don't care about them and they're in some OSX system headers which
means that this could break normal C and C++ stuff.

This introduces --generate-block / generate_block to generate these signatures,
and adds tests so that this is getting tested.
@emilio
Copy link
Contributor Author

emilio commented Oct 4, 2018

cc @flier

@emilio
Copy link
Contributor Author

emilio commented Oct 4, 2018

@bors-servo r+

  • Sorry for the self-stamp, but having OSX stuff that includes system headers broken is not an option.

@bors-servo
Copy link

📌 Commit 66ba296 has been approved by emilio

@bors-servo
Copy link

⌛ Testing commit 66ba296 with merge 35a349d...

bors-servo pushed a commit that referenced this pull request Oct 4, 2018
Puts blocks behind a switch.

Since #1378 broke a bunch of OSX builds.

Most people don't care about them and they're in some OSX system headers which
means that this could break normal C and C++ stuff.

This introduces --generate-block / generate_block to generate these signatures,
and adds tests so that this is getting tested.
@emilio emilio changed the title Puts blocks behind a switch. Put blocks behind a switch. Oct 4, 2018
@bors-servo
Copy link

☀️ Test successful - status-travis
Approved by: emilio
Pushing 35a349d to master...

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

Successfully merging this pull request may close these issues.

3 participants