Skip to content

Conversation

@jonathanpberger
Copy link

Thanks to @imajes for finding and showing me how to fix this one.

Thanks to @imajes for finding and showing me how to fix this one.
@imajes
Copy link

imajes commented Oct 20, 2016

I'm assuming here that-

a: portability across platforms isn't a problem here (not 1000% sure where fcntl is, but pretty sure it's not a problem for platforms people care about)

b: that given this wasn't required originally, and i can't find any easy google answer for whether ruby 1.8x automatically required it -- i'm assuming that @ryantate may have it wrapped into his ruby (OSX system ruby?) or something.

so, it should be safe... 👍

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.

2 participants