time_r: Fix for mingw.
authorBruno Haible <bruno@clisp.org>
Sat, 16 Nov 2019 12:13:58 +0000 (13:13 +0100)
committerBruno Haible <bruno@clisp.org>
Sat, 16 Nov 2019 12:13:58 +0000 (13:13 +0100)
Reported by Christian Biesinger <cbiesinger@google.com> in
<https://lists.gnu.org/archive/html/bug-gnulib/2019-11/msg00014.html>.

* lib/time.in.h: On mingw, include <unistd.h>.
* m4/time_r.m4 (gl_TIME_R): On mingw, include <unistd.h> before
<time.h>. Test for localtime_r in a way that works when it is defined
as an inline function.

ChangeLog
lib/time.in.h
m4/time_r.m4

index 7cc8bdde64fed39ddbb255b9e047f0caaf330d45..c01171007134a8e7633d14b98083ce6067b04b32 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,13 @@
+2019-11-16  Bruno Haible  <bruno@clisp.org>
+
+       time_r: Fix for mingw.
+       Reported by Christian Biesinger <cbiesinger@google.com> in
+       <https://lists.gnu.org/archive/html/bug-gnulib/2019-11/msg00014.html>.
+       * lib/time.in.h: On mingw, include <unistd.h>.
+       * m4/time_r.m4 (gl_TIME_R): On mingw, include <unistd.h> before
+       <time.h>. Test for localtime_r in a way that works when it is defined
+       as an inline function.
+
 2019-11-13  Bruno Haible  <bruno@clisp.org>
 
        havelib: Revert last change.
index 94e1da3a84575a8a7c49f5ac93433426c43f553f..dc0050347b7e1032554f27601973cc0b007f1858 100644 (file)
 
 # define _@GUARD_PREFIX@_TIME_H
 
+/* mingw's <time.h> provides the functions asctime_r, ctime_r, gmtime_r,
+   localtime_r only if <unistd.h> or <pthread.h> has been included before.  */
+# if defined __MINGW32__
+#  include <unistd.h>
+# endif
+
 # @INCLUDE_NEXT@ @NEXT_TIME_H@
 
 /* NetBSD 5.0 mis-defines NULL.  */
index 5caeca75093743a88dcdee647d76752950dd35dc..72cc97508a112c5fe7c9579cd772d5533c9ce75c 100644 (file)
@@ -17,19 +17,54 @@ AC_DEFUN([gl_TIME_R],
 
   dnl Some systems don't declare localtime_r() and gmtime_r() if _REENTRANT is
   dnl not defined.
-  AC_CHECK_DECLS([localtime_r], [], [], [[#include <time.h>]])
+  AC_CHECK_DECLS([localtime_r], [], [],
+    [[/* mingw's <time.h> provides the functions asctime_r, ctime_r,
+         gmtime_r, localtime_r only if <unistd.h> or <pthread.h> has
+         been included before.  */
+      #if defined __MINGW32__
+      # include <unistd.h>
+      #endif
+      #include <time.h>
+    ]])
   if test $ac_cv_have_decl_localtime_r = no; then
     HAVE_DECL_LOCALTIME_R=0
   fi
 
-  AC_CHECK_FUNCS_ONCE([localtime_r])
-  if test $ac_cv_func_localtime_r = yes; then
+  dnl We can't use AC_CHECK_FUNC here, because localtime_r() is defined as an
+  dnl inline function on mingw.
+  AC_CACHE_CHECK([for localtime_r], [gl_cv_func_localtime_r],
+    [AC_LINK_IFELSE(
+       [AC_LANG_PROGRAM(
+          [[/* mingw's <time.h> provides the functions asctime_r, ctime_r,
+               gmtime_r, localtime_r only if <unistd.h> or <pthread.h> has
+               been included before.  */
+            #if defined __MINGW32__
+            # include <unistd.h>
+            #endif
+            #include <time.h>
+          ]],
+          [[time_t a;
+            struct tm r;
+            localtime_r (&a, &r);
+          ]])
+       ],
+       [gl_cv_func_localtime_r=yes],
+       [gl_cv_func_localtime_r=no])
+    ])
+  if test $gl_cv_func_localtime_r = yes; then
     HAVE_LOCALTIME_R=1
     AC_CACHE_CHECK([whether localtime_r is compatible with its POSIX signature],
       [gl_cv_time_r_posix],
       [AC_COMPILE_IFELSE(
          [AC_LANG_PROGRAM(
-            [[#include <time.h>]],
+            [[/* mingw's <time.h> provides the functions asctime_r, ctime_r,
+                 gmtime_r, localtime_r only if <unistd.h> or <pthread.h> has
+                 been included before.  */
+              #if defined __MINGW32__
+              # include <unistd.h>
+              #endif
+              #include <time.h>
+            ]],
             [[/* We don't need to append 'restrict's to the argument types,
                  even though the POSIX signature has the 'restrict's,
                  since C99 says they can't affect type compatibility.  */