Skip to content

Future-proof the kevent ABI #1716

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

Merged
merged 1 commit into from
May 9, 2022
Merged

Future-proof the kevent ABI #1716

merged 1 commit into from
May 9, 2022

Conversation

asomers
Copy link
Member

@asomers asomers commented May 7, 2022

FreeBSD 12 changes struct kevent. For now, libc always binds to the
11-compat ABI. But that will change some day. Adjust Nix's code to
build with either struct definition.

FreeBSD 12 changes struct kevent.  For now, libc always binds to the
11-compat ABI.  But that will change some day.  Adjust Nix's code to
build with either struct definition.
@asomers asomers requested a review from rtzoeller May 7, 2022 14:13
@rtzoeller
Copy link
Collaborator

bors r+

bors bot added a commit that referenced this pull request May 9, 2022
1716: Future-proof the kevent ABI r=rtzoeller a=asomers

FreeBSD 12 changes struct kevent.  For now, libc always binds to the
11-compat ABI.  But that will change some day.  Adjust Nix's code to
build with either struct definition.

Co-authored-by: Alan Somers <[email protected]>
@bors
Copy link
Contributor

bors bot commented May 9, 2022

Build failed:

@rtzoeller
Copy link
Collaborator

bors retry

@bors bors bot merged commit db41e0b into master May 9, 2022
@asomers asomers deleted the kevent12 branch August 12, 2022 00:21
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