On Wed, 25 Aug 2010, Raphael Kubo da Costa wrote:
> On Wednesday 25 August 2010 16:30:04 Joerg Sonnenberger wrote: >> On Wed, Aug 25, 2010 at 08:32:28PM +0200, Vincent Torri wrote: >>> On Mon, 23 Aug 2010, Joerg Sonnenberger wrote: >>>> On Sun, Aug 22, 2010 at 06:08:38AM +0200, Vincent Torri wrote: >>>>>> -#if defined (__MacOSX__) || defined (__FreeBSD__) || (defined >>>>>> (__MACH__) && \ - defined (__APPLE__)) >>>>>> -# include <sys/syslimits.h> >>>>>> -#endif >>>>>> +#include <limits.h> >>>>> >>>>> Did Joerg try that on all the platforms that were in the #define ? If >>>>> no, revert this. I did ask people where the nedded constant were >>>>> located, and it was not in limits.h >>>> >>>> The original version before my change clearly doesn't work on POSIX >>>> compliant systems. For FreeBSD, sys/limits.h has been included by >>>> limits.h for at least 7 years and they did some header reorg at the >>>> time. I can't say for Darwin, but I would strongly surprised as I have >>>> enough code using PATH_MAX and co without ever including sys/limits.h. >>>> So can you demonstrate any breakage or is this just a theoretical >>>> remark? >>> >>> I was in contact with 2 mac os x users and one freebsd user. I try >>> to fixed the compilation, and limits.h was not sufficient. >> >> Not enough for WHAT? Given that Mac OS X is supposedly certified for >> POSIX compliants, that would be surprising. It would also disagree what >> I have been told today when asking exactly this question. > > FWIW, on FreeBSD at least, limits.h includes both sys/limits.h and > sys/syslimits.h (the latter only if __POSIX_VISIBLE is defined). All versions ? Vincent > See > http://svn.freebsd.org/viewvc/base/stable/8/include/limits.h?revision=196045&view=markup > > -- > Raphael Kubo da Costa > ProFUSION embedded systems > http://profusion.mobi > > ------------------------------------------------------------------------------ > Sell apps to millions through the Intel(R) Atom(Tm) Developer Program > Be part of this innovative community and reach millions of netbook users > worldwide. Take advantage of special opportunities to increase revenue and > speed time-to-market. Join now, and jumpstart your future. > http://p.sf.net/sfu/intel-atom-d2d > _______________________________________________ > enlightenment-devel mailing list > enlightenment-devel@lists.sourceforge.net > https://lists.sourceforge.net/lists/listinfo/enlightenment-devel > > ------------------------------------------------------------------------------ Sell apps to millions through the Intel(R) Atom(Tm) Developer Program Be part of this innovative community and reach millions of netbook users worldwide. Take advantage of special opportunities to increase revenue and speed time-to-market. Join now, and jumpstart your future. http://p.sf.net/sfu/intel-atom-d2d _______________________________________________ enlightenment-devel mailing list enlightenment-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
