stdlib: work around MirBSD WEXITSTATUS bug

* lib/stdlib.in.h (includes): Guarantee WEXITSTATUS.
* modules/stdlib (Depends-on): Add sys_wait.
* tests/test-sys_wait.c (main): Enhance test.
* tests/test-stdlib.c (main): Likewise.
* doc/posix-headers/stdlib.texi (stdlib.h): Document the bug.

Signed-off-by: Eric Blake <eblake@redhat.com>

diff --git a/ChangeLog b/ChangeLog
index ffbb1c03e2930a3d272dacdd155d3fdcda7a5cee..bacfce312680e037e9edfde42b8e87d58a06c044 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,12 @@
 2010-09-15  Eric Blake  <eblake@redhat.com>
 
+       stdlib: work around MirBSD WEXITSTATUS bug
+       * lib/stdlib.in.h (includes): Guarantee WEXITSTATUS.
+       * modules/stdlib (Depends-on): Add sys_wait.
+       * tests/test-sys_wait.c (main): Enhance test.
+       * tests/test-stdlib.c (main): Likewise.
+       * doc/posix-headers/stdlib.texi (stdlib.h): Document the bug.
+
        docs: mention MacOS issue with WEXITSTATUS(constant)
        * doc/posix-headers/sys_wait.texi (sys/wait.h): Document the
        issue.

diff --git a/doc/posix-headers/stdlib.texi b/doc/posix-headers/stdlib.texi
index e1d6aba7d0d1542976efc111eacf041502d7393e..f400f62749f5f7b7c320a3f44934ad4ef42f1622 100644 (file)
--- a/doc/posix-headers/stdlib.texi
+++ b/doc/posix-headers/stdlib.texi
@@ -17,6 +17,10 @@ The macro @code{EXIT_FAILURE} is incorrectly defined on Tandem/NSK.
 Some platforms provide a @code{NULL} macro that cannot be used in arbitrary
 expressions:
 NetBSD 5.0
+
+@item
+Some platforms fail to provide @code{WEXITSTATUS} and friends:
+MirBSD 10.
 @end itemize
 
 Portability problems not fixed by Gnulib:

diff --git a/lib/stdlib.in.h b/lib/stdlib.in.h
index d4ac4690305e702548d04ab816ccf66c2056e7b4..0c1c1659352b51578dff4ec7dd457f7238cd9012 100644 (file)
--- a/lib/stdlib.in.h
+++ b/lib/stdlib.in.h
@@ -38,6 +38,11 @@
 /* NetBSD 5.0 mis-defines NULL.  */
 #include <stddef.h>
 
+/* MirBSD 10 fails to define WEXITSTATUS.  */
+#ifndef WEXITSTATUS
+# include <sys/wait.h>
+#endif
+
 /* Solaris declares getloadavg() in <sys/loadavg.h>.  */
 #if (@GNULIB_GETLOADAVG@ || defined GNULIB_POSIXCHECK) && @HAVE_SYS_LOADAVG_H@
 # include <sys/loadavg.h>

diff --git a/modules/stdlib b/modules/stdlib
index 489f70a1a51333a54a8da9119e9a7db0d2023eac..f822aa3ebe64a472c6150f69904c376adbdbfc8a 100644 (file)
--- a/modules/stdlib
+++ b/modules/stdlib
@@ -11,6 +11,7 @@ c++defs
 include_next
 stddef
 stdint
+sys_wait
 unistd
 warn-on-use
 

diff --git a/tests/test-stdlib.c b/tests/test-stdlib.c
index 4bd8715f3d619615d80421bdb4f201cdc64e202d..8066b0525ddecf0ea64552a541e77b190102db5c 100644 (file)
--- a/tests/test-stdlib.c
+++ b/tests/test-stdlib.c
@@ -22,7 +22,17 @@
 
 #include "verify.h"
 
-int exitcode;
+/* Check that EXIT_SUCCESS is 0, per POSIX.  */
+static int exitcode = EXIT_SUCCESS;
+#if EXIT_SUCCESS
+"oops"
+#endif
+
+/* Check for GNU value (not guaranteed by POSIX, but is guaranteed by
+   gnulib).  */
+#if EXIT_FAILURE != 1
+"oops"
+#endif
 
 /* Check that NULL can be passed through varargs as a pointer type,
    per POSIX 2008.  */
@@ -31,13 +41,23 @@ verify (sizeof NULL == sizeof (void *));
 int
 main (void)
 {
-  /* Check that some macros are defined and different integer constants.  */
-  switch (exitcode)
+  /* Check subset of <sys/wait.h> macros that must be visible here.
+     Note that some of these macros are only portable when operating
+     on an lvalue.  */
+  int i;
+  for (i = 0; i < 0x10000; i = (i ? i << 1 : 1))
+    if (!!WIFSIGNALED (i) + !!WIFEXITED (i) + !!WIFSTOPPED (i) != 1)
+      return 1;
+  i = WEXITSTATUS (i) + WSTOPSIG (i) + WTERMSIG (i);
+
+  switch (i)
     {
-    case EXIT_SUCCESS:
-    case EXIT_FAILURE:
+#if 0
+  /* Gnulib doesn't guarantee these, yet.  */
+    case WNOHANG:
+    case WUNTRACED:
+#endif
       break;
     }
-
   return 0;
 }

diff --git a/tests/test-sys_wait.c b/tests/test-sys_wait.c
index 8c0170ed9b875149e38eab36eae63efcc4945c85..917de0dc692980da966592ca498412ee8c98a96d 100644 (file)
--- a/tests/test-sys_wait.c
+++ b/tests/test-sys_wait.c
@@ -26,5 +26,26 @@ static pid_t a;
 int
 main (void)
 {
-  return a;
+  /* Check for existence of required macros.  Note that we document
+     that these are safe only on lvalues.  */
+  int i;
+  for (i = 0; i < 0x10000; i = (i ? i << 1 : 1))
+    if (!!WIFSIGNALED (i) + !!WIFEXITED (i) + !!WIFSTOPPED (i) != 1)
+      return 1;
+  i = WEXITSTATUS (i) + WSTOPSIG (i) + WTERMSIG (i);
+
+  switch (i)
+    {
+#if 0
+  /* Gnulib doesn't guarantee these, yet.  */
+    case WCONTINUED:
+    case WNOHANG:
+    case WUNTRACED:
+    case WEXITED:
+    case WNOWAIT:
+    case WSTOPPED:
+#endif
+      break;
+    }
+  return a ? i : 0;
 }