]> xenbits.xensource.com Git - libvirt.git/commitdiff
examples: Work around lack of mingw localtime_r()
authorEric Blake <eblake@redhat.com>
Tue, 8 Jan 2019 19:41:28 +0000 (13:41 -0600)
committerEric Blake <eblake@redhat.com>
Tue, 8 Jan 2019 20:47:21 +0000 (14:47 -0600)
mingw lacks localtime_r(); we were getting it from gnulib. But since
commit acf522e8 stopped linking examples against gnulib, we are
getting a build failure. Keep the examples standalone, and work
around mingw by using the non-reentrant localtime() (safe since our
examples are single-threaded), and add a necessary exemption to our
syntax check.

Signed-off-by: Eric Blake <eblake@redhat.com>
Reviewed-by: John Ferlan <jferlan@redhat.com>
cfg.mk
examples/admin/client_info.c
examples/admin/list_clients.c

diff --git a/cfg.mk b/cfg.mk
index d9a8a44dc437d0355363bfbd03719895713003ad..a75bfef86e5817057662cecc9c4087357b2d7a0d 100644 (file)
--- a/cfg.mk
+++ b/cfg.mk
@@ -1233,7 +1233,7 @@ exclude_file_name_regexp--sc_prohibit_newline_at_end_of_diagnostic = \
   ^src/rpc/gendispatch\.pl$$
 
 exclude_file_name_regexp--sc_prohibit_nonreentrant = \
-  ^((po|tests)/|docs/.*(py|js|html\.in)|run.in$$|tools/wireshark/util/genxdrstub\.pl$$)
+  ^((po|tests|examples/admin)/|docs/.*(py|js|html\.in)|run.in$$|tools/wireshark/util/genxdrstub\.pl$$)
 
 exclude_file_name_regexp--sc_prohibit_select = \
        ^cfg\.mk$$
index 2c46ccf5145a591fe214304e00e3615a23a12d4d..f3f62a656b45b5e3dd4d40f8c9896af0542ba941 100644 (file)
@@ -30,9 +30,13 @@ exampleGetTimeStr(time_t then)
 {
     char *ret = NULL;
     struct tm timeinfo;
+    struct tm *timeinfop;
 
-    if (!localtime_r(&then, &timeinfo))
+    /* localtime_r() is smarter, but since mingw lacks it and this
+     * example is single-threaded, we can get away with localtime */
+    if (!(timeinfop = localtime(&then)))
         return NULL;
+    timeinfo = *timeinfop;
 
     if (!(ret = calloc(64, sizeof(char))))
         return NULL;
index 15205a7bd1e01e6de0fc4c2d7420d366c01c4857..5cf8e4c2a6af1a0db14bdaeef6bc34e751d8849f 100644 (file)
@@ -28,9 +28,13 @@ exampleGetTimeStr(time_t then)
 {
     char *ret = NULL;
     struct tm timeinfo;
+    struct tm *timeinfop;
 
-    if (!localtime_r(&then, &timeinfo))
+    /* localtime_r() is smarter, but since mingw lacks it and this
+     * example is single-threaded, we can get away with localtime */
+    if (!(timeinfop = localtime(&then)))
         return NULL;
+    timeinfo = *timeinfop;
 
     if (!(ret = calloc(64, sizeof(char))))
         return NULL;