Re: kqueue.c and <sys/file.h>

[Michael Albinus]

Jeremie Courreges-Anglas <address@hidden> writes:

> Hi,

Hi Jeremie,

> While sys/file.h has been around since some time now, it is not part of
> the POSIX standard (or any standard that I know of), and it looks like
> most of its consumers really want to include <fcntl.h> instead to access
> the various O_* #defines used by open(2).
>
> Within Emacs, src/kqueue.c seems to be the only source file that depends
> on sys/file.h to build on OpenBSD.  As discussed with Michael Albinus,
> I'd suggest using fcntl.h there (trivial diff below).  I guess it should
> be tested on other systems that provide kqueue(2) (*BSD, OSX, ?) but
> I don't expect fallout.

Thanks for this! Pls provide also a ChangeLog entry with reasoning for
this patch.

Let's wait a week or so. If nobody complains, I'll commit the patch to
the master branch then, in your name.

Best regards, Michael.