> 2011-09-11 Bruno Haible <br...@clisp.org> > > Ensure pid_t gets defined. > * m4/time_h.m4 (gl_HEADER_TIME_H_BODY): Likewise. > * tests/test-time.c: Likewise. > * doc/posix-headers/time.texi: Likewise.
The part regarding <time.h> was unncessary, since POSIX:2008 want pid_t in <time.h> only under "CPT" shading. The test actually failed to compile on MacOS X 10.5, FreeBSD 6.4, OpenBSD 4.9, mingw. I'm undoing this: 2011-09-20 Bruno Haible <br...@clisp.org> time tests: Don't require pid_t. * doc/posix-headers/time.texi: Revert last change. * m4/time_h.m4 (gl_HEADER_TIME_H_BODY): Revert last change. * tests/test-time.c: Comment out the check for pid_t. --- tests/test-time.c.orig Wed Sep 21 01:02:26 2011 +++ tests/test-time.c Wed Sep 21 01:01:45 2011 @@ -24,7 +24,11 @@ /* Check that the types are all defined. */ struct timespec t1; +#if 0 +/* POSIX:2008 does not require pid_t in <time.h> unconditionally, and indeed + it's missing on MacOS X 10.5, FreeBSD 6.4, OpenBSD 4.9, mingw. */ pid_t t2; +#endif /* Check that NULL can be passed through varargs as a pointer type, per POSIX 2008. */ --- doc/posix-headers/time.texi.orig Wed Sep 21 01:07:27 2011 +++ doc/posix-headers/time.texi Wed Sep 21 01:07:04 2011 @@ -11,10 +11,6 @@ @samp{struct timespec} is not defined on some platforms. @item -The type @code{pid_t} is not defined on some platforms: -MSVC 9. - -@item Some platforms provide a @code{NULL} macro that cannot be used in arbitrary expressions: NetBSD 5.0 --- m4/time_h.m4.orig Wed Sep 21 01:07:27 2011 +++ m4/time_h.m4 Wed Sep 21 01:06:17 2011 @@ -2,7 +2,7 @@ # Copyright (C) 2000-2001, 2003-2007, 2009-2011 Free Software Foundation, Inc. -# serial 5 +# serial 6 # This file is free software; the Free Software Foundation # gives unlimited permission to copy and/or distribute it, @@ -23,9 +23,6 @@ AC_REQUIRE([gl_HEADER_TIME_H_DEFAULTS]) gl_NEXT_HEADERS([time.h]) AC_REQUIRE([gl_CHECK_TYPE_STRUCT_TIMESPEC]) - - dnl Ensure the type pid_t gets defined. - AC_REQUIRE([AC_TYPE_PID_T]) ]) dnl Define HAVE_STRUCT_TIMESPEC if `struct timespec' is declared -- In memoriam Pierre Goldman <http://en.wikipedia.org/wiki/Pierre_Goldman>