]> Savannah Git Hosting - gnulib.git/commitdiff
utimensat: remove FIXME for old Linux kernels
authorPaul Eggert <eggert@cs.ucla.edu>
Tue, 1 Nov 2016 15:55:17 +0000 (08:55 -0700)
committerPaul Eggert <eggert@cs.ucla.edu>
Tue, 1 Nov 2016 15:55:47 +0000 (08:55 -0700)
* lib/utimensat.c (rpl_utimensat): Update FIXME comment.
* m4/utimensat.m4 (gl_FUNC_UTIMENSAT): Belatedy do a "simplify
this in 2012" FIXME, by assuming the file system bug is absent
unless demonstrated to be present.  We no longer need to worry
about Linux kernel 2.6.32 when building with newer kernels.

ChangeLog
lib/utimensat.c
m4/utimensat.m4

index c8ac92c8f0aea10560f5df8edadedb6624004cdd..999dc0418a1c0a8a1218e08246b81c62c660e8c3 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,12 @@
+2016-11-01  Paul Eggert  <eggert@cs.ucla.edu>
+
+       utimensat: remove FIXME for old Linux kernels
+       * lib/utimensat.c (rpl_utimensat): Update FIXME comment.
+       * m4/utimensat.m4 (gl_FUNC_UTIMENSAT): Belatedy do a "simplify
+       this in 2012" FIXME, by assuming the file system bug is absent
+       unless demonstrated to be present.  We no longer need to worry
+       about Linux kernel 2.6.32 when building with newer kernels.
+
 2016-10-16  Bruno Haible  <bruno@clisp.org>
 
        qsort_r: Fix macrology for platforms that lack the function.
index ae4da8f7d9252608a6101ab208c5e1bd592db7fe..7ca3e4b39447236259db0e94cc57b194bad8934e 100644 (file)
@@ -69,8 +69,8 @@ rpl_utimensat (int fd, char const *file, struct timespec const times[2],
 
          The same bug occurs in Solaris 11.1 (Apr 2013).
 
-         FIXME: Simplify this for Linux in 2016 and for Solaris in
-         2024, when file system bugs are no longer common.  */
+         FIXME: Simplify this in 2024, when these file system bugs are
+         no longer common on Gnulib target platforms.  */
       if (times && (times[0].tv_nsec == UTIME_OMIT
                     || times[1].tv_nsec == UTIME_OMIT))
         {
index 550cd23c73b5bed7fe38a9e3d97d5609550079cb..d9a4cd39bf930f4d0a20abeb7b88adfd8d42890d 100644 (file)
@@ -1,4 +1,4 @@
-# serial 5
+# serial 6
 # See if we need to provide utimensat replacement.
 
 dnl Copyright (C) 2009-2016 Free Software Foundation, Inc.
@@ -51,20 +51,10 @@ AC_DEFUN([gl_FUNC_UTIMENSAT],
               }
               return result;
             ]])],
-dnl FIXME: simplify this in 2012, when file system bugs are no longer common
-         [AC_COMPILE_IFELSE([AC_LANG_PROGRAM([[
-#ifdef __linux__
-/* The Linux kernel added utimensat in 2.6.22, but has bugs with UTIME_OMIT
-   in several file systems as recently as 2.6.32.  Always replace utimensat
-   to support older kernels.  */
-choke me
-#endif
-      ]])],
-           [gl_cv_func_utimensat_works=yes],
-           [gl_cv_func_utimensat_works="needs runtime check"])],
+         [gl_cv_func_utimensat_works=yes],
          [gl_cv_func_utimensat_works=no],
-         [gl_cv_func_utimensat_works="guessing no"])])
-    if test "$gl_cv_func_utimensat_works" != yes; then
+         [gl_cv_func_utimensat_works="guessing yes"])])
+    if test "$gl_cv_func_utimensat_works" = no; then
       REPLACE_UTIMENSAT=1
     fi
   fi