Switch to utimensat for newer POSIX versions
authorRosen Penev <rosenp@gmail.com>
Fri, 9 Aug 2019 20:01:05 +0000 (13:01 -0700)
committerErik de Castro Lopo <erikd@mega-nerd.com>
Mon, 19 Aug 2019 20:49:13 +0000 (06:49 +1000)
Some libcs like uClibc-ng can optionally disable deprecated functions.
utime is one of them. When done so, both the header and the function go
missing.

This fixes flac_utime to work in such a situation.

include/share/compat.h
src/libFLAC/metadata_iterators.c
src/share/grabbag/file.c
src/test_libFLAC++/metadata_manip.cpp
src/test_libFLAC/metadata_manip.c

index f304165..c28054b 100644 (file)
 #include <sys/utime.h> /* for utime() */
 #endif
 #else
+#if defined(_POSIX_C_SOURCE) && (_POSIX_C_SOURCE >= 200809L)
+#include <fcntl.h>
+#else
 #include <sys/types.h> /* some flavors of BSD (like OS X) require this to get time_t */
 #include <utime.h> /* for utime() */
 #endif
+#endif
 
 #if defined _MSC_VER
 #  if _MSC_VER >= 1800
 
 #define flac_fopen fopen
 #define flac_chmod chmod
-#define flac_utime utime
 #define flac_unlink unlink
 #define flac_rename rename
 #define flac_stat stat
 
+#if defined(_POSIX_C_SOURCE) && (_POSIX_C_SOURCE >= 200809L)
+#define flac_utime(a, b) utimensat (AT_FDCWD, a, *b, 0)
+#else
+#define flac_utime utime
+#endif
 #endif
 
 #ifdef _WIN32
index 352a6c7..fca2c15 100644 (file)
@@ -3422,13 +3422,18 @@ FLAC__bool get_file_stats_(const char *filename, struct flac_stat_s *stats)
 
 void set_file_stats_(const char *filename, struct flac_stat_s *stats)
 {
-       struct utimbuf srctime;
-
        FLAC__ASSERT(0 != filename);
        FLAC__ASSERT(0 != stats);
 
+#if defined(_POSIX_C_SOURCE) && (_POSIX_C_SOURCE >= 200809L)
+       struct timespec srctime[2] = {};
+       srctime[0].tv_sec = stats->st_atime;
+       srctime[1].tv_sec = stats->st_mtime;
+#else
+       struct utimbuf srctime;
        srctime.actime = stats->st_atime;
        srctime.modtime = stats->st_mtime;
+#endif
        (void)flac_chmod(filename, stats->st_mode);
        (void)flac_utime(filename, &srctime);
 #if !defined _MSC_VER && !defined __BORLANDC__ && !defined __MINGW32__
index 2c67beb..5f3bc4e 100644 (file)
@@ -27,7 +27,6 @@
 #include <fcntl.h> /* for _O_BINARY */
 #else
 #include <sys/types.h> /* some flavors of BSD (like OS X) require this to get time_t */
-#include <utime.h> /* for utime() */
 #endif
 #if defined __EMX__
 #include <io.h> /* for setmode(), O_BINARY */
 void grabbag__file_copy_metadata(const char *srcpath, const char *destpath)
 {
        struct flac_stat_s srcstat;
-       struct utimbuf srctime;
 
        if(0 == flac_stat(srcpath, &srcstat)) {
+#if defined(_POSIX_C_SOURCE) && (_POSIX_C_SOURCE >= 200809L)
+               struct timespec srctime[2] = {};
+               srctime[0].tv_sec = srcstat.st_atime;
+               srctime[1].tv_sec = srcstat.st_mtime;
+#else
+               struct utimbuf srctime;
                srctime.actime = srcstat.st_atime;
                srctime.modtime = srcstat.st_mtime;
+#endif
                (void)flac_chmod(destpath, srcstat.st_mode);
                (void)flac_utime(destpath, &srctime);
        }
index 93b6e51..ab98aa1 100644 (file)
@@ -27,8 +27,6 @@
 #include <sys/types.h> /* some flavors of BSD (like OS X) require this to get time_t */
 #ifdef _MSC_VER
 #include <sys/utime.h>
-#else
-#include <utime.h> /* for utime() */
 #endif
 #if !defined _MSC_VER && !defined __MINGW32__ && !defined __EMX__
 #include <unistd.h> /* for chown(), unlink() */
@@ -271,13 +269,18 @@ bool get_file_stats_(const char *filename, struct flac_stat_s *stats)
 
 void set_file_stats_(const char *filename, struct flac_stat_s *stats)
 {
-       struct utimbuf srctime;
-
        FLAC__ASSERT(0 != filename);
        FLAC__ASSERT(0 != stats);
 
+#if defined(_POSIX_C_SOURCE) && (_POSIX_C_SOURCE >= 200809L)
+       struct timespec srctime[2] = {};
+       srctime[0].tv_sec = stats->st_atime;
+       srctime[1].tv_sec = stats->st_mtime;
+#else
+       struct utimbuf srctime;
        srctime.actime = stats->st_atime;
        srctime.modtime = stats->st_mtime;
+#endif
        (void)flac_chmod(filename, stats->st_mode);
        (void)flac_utime(filename, &srctime);
 #if !defined _MSC_VER && !defined __MINGW32__ && !defined __EMX__
index b4bf065..da10d85 100644 (file)
@@ -29,7 +29,6 @@
 #include <io.h> /* for chmod() */
 #else
 #include <sys/types.h> /* some flavors of BSD (like OS X) require this to get time_t */
-#include <utime.h> /* for utime() */
 #include <unistd.h> /* for chown(), unlink() */
 #endif
 #include <sys/stat.h> /* for stat(), maybe chmod() */
@@ -256,13 +255,18 @@ static FLAC__bool get_file_stats_(const char *filename, struct flac_stat_s *stat
 
 static void set_file_stats_(const char *filename, struct flac_stat_s *stats)
 {
-       struct utimbuf srctime;
-
        FLAC__ASSERT(0 != filename);
        FLAC__ASSERT(0 != stats);
 
+#if defined(_POSIX_C_SOURCE) && (_POSIX_C_SOURCE >= 200809L)
+       struct timespec srctime[2] = {};
+       srctime[0].tv_sec = stats->st_atime;
+       srctime[1].tv_sec = stats->st_mtime;
+#else
+       struct utimbuf srctime;
        srctime.actime = stats->st_atime;
        srctime.modtime = stats->st_mtime;
+#endif
        (void)flac_chmod(filename, stats->st_mode);
        (void)flac_utime(filename, &srctime);
 #if !defined _MSC_VER && !defined __MINGW32__