On 05/04/2010 01:42 AM, Brett Lymn wrote:
> On Mon, May 03, 2010 at 05:08:29PM +0200, Silamael wrote:
>
>> Fix include order to ensure that FD_SETSIZE from the compat/fdsetsize.h is set
>> before it is set by sys/select.h (included by stdlib.h).
>>
>
> To be strictly correct about this, the problem is really an OpenBSD
> one. There should not be an order dependence on the inclusion of
> system headers. You could try logging it as a bug on OpenBSD - good
> luck with that.
>
No, i dont think thats an OpenBSD only problem. The ordering of include
files in this authentication helper always would first include the
system headers - also on Linux, MacOS X, whatever - and then redefine
FD_SETSIZE with some own value given to configure. In my opinion this
compiler warning will be happen on most systems except the ones for
which compat/fdsetsize.h has some hacks to prevent setting FD_SETSIZE.
-- Matthias
Received on Tue May 04 2010 - 05:42:42 MDT
This archive was generated by hypermail 2.2.0 : Tue May 04 2010 - 12:00:03 MDT