Re: [PATCH] autoconf: Add test for sys/select.h header file

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Jan 25, 2008 at 10:25:33PM -0800, Junio C Hamano wrote:
> The issue is not really about "do we have <sys/select.h>?" but
> is about "do we need to include <sys/select.h> in order to
> obtain declaration of select(2)?" so this is covering only a

This is true.  Actually HP-UX 11.23 for instance has sys/select.h but this is
not intended to be included to get the select call but this is intended to be
included by kernel drivers that implement the interface the kernel needs to
_provide_ the select call.  In that case the test determines that sys/select.h
exists but it does not notice that it is not the header to include.  It does
not hurt though since including sys/select.h outside of a kernel driver is a
no-op and we include sys/time.h which is really needed anyway.

> half of the issue.  But I guess we will find out if the system
> does not offer select(2) whether <sys/select.h> is included or
> not, so this is not too bad.
> 
> Will apply.  I do not think it would have problems but even if
> it does it is small and easy to revert before the final.

At least on the systems I know it does not hurt even when it does not always
exactly what it is intended to do.

Robert

-- 
Robert Schiele
Dipl.-Wirtsch.informatiker	mailto:rschiele@xxxxxxxxx

"Quidquid latine dictum sit, altum sonatur."

Attachment: pgpm8yi0Ow7fp.pgp
Description: PGP signature


[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]

  Powered by Linux