Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Disable epoll support on Solaris. #1208

Merged
merged 5 commits into from
Nov 5, 2024
Merged

Conversation

sunfishcode
Copy link
Member

It appears Solaris doesn't support epoll; only Illumos does. And as of rust-lang/libc#3864, the libc crate won't declare the epoll libc interface on Solaris. So disable epoll support on Solaris in rustix too.

It appears Solaris doesn't support epoll; only Illumos does. And as
of rust-lang/libc#3864, the libc crate won't declare the epoll libc
interface on Solaris. So disable epoll support on Solaris in rustix
too.
@sunfishcode sunfishcode merged commit 53529ac into main Nov 5, 2024
45 checks passed
@sunfishcode sunfishcode deleted the sunfishcode/no-epoll-solaris branch November 5, 2024 20:57
sunfishcode added a commit that referenced this pull request Nov 10, 2024
* Disable epoll support on Solaris.

It appears Solaris doesn't support epoll; only Illumos does. And as
of rust-lang/libc#3864, the libc crate won't declare the epoll libc
interface on Solaris. So disable epoll support on Solaris in rustix
too.

* Add epoll documentation links for illumos.

* Add a documentation reference for ptsname for illumos.

* Enable the epoll tests on illumos and redox.

* Enable the openpty tests on illumos.
sunfishcode added a commit that referenced this pull request Nov 10, 2024
* Disable epoll support on Solaris.

It appears Solaris doesn't support epoll; only Illumos does. And as
of rust-lang/libc#3864, the libc crate won't declare the epoll libc
interface on Solaris. So disable epoll support on Solaris in rustix
too.

* Add epoll documentation links for illumos.

* Add a documentation reference for ptsname for illumos.

* Enable the epoll tests on illumos and redox.

* Enable the openpty tests on illumos.
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant