Skip to content
This repository has been archived by the owner on Oct 2, 2024. It is now read-only.

configure.ac: add missed fnctl.h #1887

Merged
merged 1 commit into from
Mar 28, 2024
Merged

configure.ac: add missed fnctl.h #1887

merged 1 commit into from
Mar 28, 2024

Conversation

reidpr
Copy link
Collaborator

@reidpr reidpr commented Mar 27, 2024

open(2) is documented to require fnctl.h. We weren’t including it explicitly, but it worked, I assume because we got fnctl.h indirectly. However this is apparently not true on Ubuntu 22.04, and regardless we should include the documented file.

@reidpr reidpr added this to the 0.38 milestone Mar 27, 2024
@reidpr reidpr self-assigned this Mar 27, 2024
@reidpr reidpr requested a review from lucaudill March 28, 2024 15:43
Copy link
Collaborator

@lucaudill lucaudill left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@reidpr reidpr merged commit 88e9a71 into master Mar 28, 2024
6 checks passed
@reidpr reidpr deleted the configure-fnctl branch March 28, 2024 21:01
kchilleri pushed a commit that referenced this pull request May 8, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants