Skip to content

Conversation

@Susurrus
Copy link
Contributor

@Susurrus Susurrus commented Jul 2, 2017

Missing a few constants across all nix-supported platforms (see nix-rust/nix#636) so this adds them. This is still a work in progress as I wanted to make sure I didn't break anything doing most of the fixes. I'll come back and finish this up later.

@alexcrichton
Copy link
Member

@bors: r+

@bors
Copy link
Contributor

bors commented Jul 2, 2017

📋 Looks like this PR is still in progress, ignoring approval

@Susurrus Susurrus force-pushed the more_socket_constants branch from 24495bd to 2a661b3 Compare July 2, 2017 23:37
@Susurrus Susurrus force-pushed the more_socket_constants branch from 2a661b3 to 2a27242 Compare July 2, 2017 23:45
@Susurrus Susurrus changed the title WIP: Add socket constants for more platforms Add socket constants for more platforms Jul 3, 2017
@Susurrus
Copy link
Contributor Author

Susurrus commented Jul 3, 2017

@alexcrichton Alright, this is actually ready now!

@alexcrichton
Copy link
Member

@bors: r+

@bors
Copy link
Contributor

bors commented Jul 3, 2017

📌 Commit 2a27242 has been approved by alexcrichton

@bors
Copy link
Contributor

bors commented Jul 3, 2017

⌛ Testing commit 2a27242 with merge 158d778...

bors added a commit that referenced this pull request Jul 3, 2017
Add socket constants for more platforms

Missing a few constants across all nix-supported platforms (see nix-rust/nix#636) so this adds them. This is still a work in progress as I wanted to make sure I didn't break anything doing most of the fixes. I'll come back and finish this up later.
@bors
Copy link
Contributor

bors commented Jul 3, 2017

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

@bors bors merged commit 2a27242 into rust-lang:master Jul 3, 2017
@Susurrus Susurrus deleted the more_socket_constants branch July 3, 2017 02:06
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