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

Remove 1.8-incompatible changes to #setsockopt #2043

Merged
merged 1 commit into from Nov 20, 2012
Merged

Remove 1.8-incompatible changes to #setsockopt #2043

merged 1 commit into from Nov 20, 2012

Conversation

warrenseen
Copy link
Contributor

This commit corrects changes mistakenly added to lib/18/socket.rb.
It also adds ruby version guards to ensure the specs only run against 1.9
implementations.

This commit corrects changes mistakenly added to lib/18/socket.rb.
It also adds ruby version guards to ensure the specs only run against 1.9
implementations.
@warrenseen
Copy link
Contributor Author

@brixen Really sorry, was hoping to get this fix into the #2040 pull request before you merged it. This now accurately reflects behaviour on MRI 1.8

dbussink added a commit that referenced this pull request Nov 20, 2012
Remove 1.8-incompatible changes to #setsockopt
@dbussink dbussink merged commit 828b704 into rubinius:master Nov 20, 2012
@frodsan
Copy link
Contributor

frodsan commented Nov 20, 2012

@warrenseen Thanks! 👍

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.

None yet

3 participants