> 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>

Reply via email to