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

Add missing termios constants for Mac #640

Merged
merged 1 commit into from
Jul 3, 2017

Conversation

Susurrus
Copy link
Contributor

@Susurrus Susurrus commented Jul 3, 2017

Didn't add these the first time around, because it says they're unimplemented, but so are the NL0 and related constants I added. This also makes testing a little easier having these constants available, and they are in the termios headers on Mac.

@Susurrus
Copy link
Contributor Author

Susurrus commented Jul 3, 2017

Alright, tests pass and this should be GTM.

@alexcrichton
Copy link
Member

@bors: r+

@bors
Copy link
Contributor

bors commented Jul 3, 2017

📌 Commit cab41d7 has been approved by alexcrichton

@bors
Copy link
Contributor

bors commented Jul 3, 2017

⌛ Testing commit cab41d7 with merge 3acddc7...

bors added a commit that referenced this pull request Jul 3, 2017
Add missing termios constants for Mac

Didn't add these the first time around, because it says they're unimplemented, but so are the `NL0` and related constants I added. This also makes testing a little easier having these constants available, and they are in the termios headers on Mac.
@bors
Copy link
Contributor

bors commented Jul 3, 2017

☀️ Test successful - status-appveyor, status-travis
Approved by: alexcrichton
Pushing 3acddc7 to master...

@bors bors merged commit cab41d7 into rust-lang:master Jul 3, 2017
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