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
