-
Notifications
You must be signed in to change notification settings - Fork 630
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
Fix build on OpenBSD #690
Fix build on OpenBSD #690
Conversation
Looks good! Would you mind adjusting indentation from spaces to tabs to match the surrounding code? |
If EPROTO does not exist define it as EINTR
Ah yes, did not see that. Should be fine now. Have been running the change for a week and no problems so far (on 1.0.9.1). |
Wait, so how'd one go about installing EventMachine on OpenBSD? Tried |
Nevermind, found http://stackoverflow.com/questions/2577346/how-to-install-gem-from-github-source and seems I can do |
Yep, that'll tide you over! |
EventMachine 1.2.0 is now released! |
Hi guys, any chance of backporting the fix to the 1.0.9.* branch? Currently Sensu 0.22.2 still requires eventmachine 1.0.9.1 - it doesn't seem to work with 1.2, but this simple patch against the tag
Would it be possible to do another release in the 1.0.9 branch? I have created an issue with Sensu, asking if they'd bump their requirement. |
Definitely, I'll work on it this week. |
Sensu has moved to eventmachine 1.2 in a new release, I believe we're good here. |
If EPROTO does not exist define it as EINTR
OpenBSD does not have EPROTO: https://sourceforge.net/p/sshguard/mailman/message/21247557/
Works, tried on 1.2.dev.2 and 1.0.9.1.