-
Notifications
You must be signed in to change notification settings - Fork 24
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 Darwin support #34
base: master
Are you sure you want to change the base?
Conversation
Now I can confirm this works on macOS/iOS, but testsuites cannot be compiled since it was using many calls that not exist on Darwin(e.g. |
Some tests failed, and it stuck at test 23
The implementations of |
I think I need some help on this, I have made tests compile, but many of them failed, I'm not sure which part causing problems
|
Thanks for you work on this! I actually tried to port this library to macOS last year, but I was discouraged by macOS' poor support for recent (and not so recent) POSIX features, and the fact that it's a bit hard for me to test this. I noticed I hadn't pushed this work yet (shame on me), but now I rebased my old branch and pushed it into the Back then I got the I think we should also look at getting a CI build for macOS running, at least for x64. GitHub Actions should support this. |
shall I rebase my current darwin branch and do pr on your macos branch? |
Yes, that would be great! |
Found by running the tests with -fsanitize=address
Some tests skipped due to different behavior with
Consider switch to kevent64 on Darwin
|
Seems nothing to change for now except some skipped tests |
Putting the definition of |
then we probably have to create a sys/time.h like that
|
merge recent upstream changes
kqueue1()
since Darwin forced close-on-exec for its file descriptors, I'm not sure if this going to affect epoll functions