]> Savannah Git Hosting - gnulib.git/commitdiff
nstrftime: remove dependency on hard-locale
authorPaul Eggert <eggert@cs.ucla.edu>
Tue, 4 Jun 2024 05:00:59 +0000 (22:00 -0700)
committerPaul Eggert <eggert@cs.ucla.edu>
Tue, 4 Jun 2024 05:04:45 +0000 (22:04 -0700)
GNU Emacs would rather keep dependencies to a minimum,
and hard-locale brings in some other stuff.
* lib/strftime.c: Include locale.h only if (USE_C_LOCALE &&
HAVE_STRFTIME_L) || ((defined __NetBSD__ || defined __sun) &&
REQUIRE_GNUISH_STRFTIME_AM_PM). Do not include hard-locale.h.
(libc_hidden_def): On native MS-Windows, use the C locale %c
format regardless of language.  I hope that’s good enough.
If not, let’s try to think of a way of solving the problem
that doesn’t require using hard_locale.
* modules/nstrftime (Depends-on): Remove hard-locale.
* modules/nstrftime-tests (test_nstrftime_LDADD):
Remove $(HARD_LOCALE_LIB).
* tests/test-nstrftime.h (locales_test):
Add a test that I hope works on French MS-Windows.

ChangeLog
lib/strftime.c
modules/nstrftime
modules/nstrftime-tests
tests/test-nstrftime.h

index bd38e2a79d761180da30369944e37c7dc13552dd..787edff23d61b2bd3e740e5e9bce5d519fe9fa07 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,21 @@
+2024-06-03  Paul Eggert  <eggert@cs.ucla.edu>
+
+       nstrftime: remove dependency on hard-locale
+       GNU Emacs would rather keep dependencies to a minimum,
+       and hard-locale brings in some other stuff.
+       * lib/strftime.c: Include locale.h only if (USE_C_LOCALE &&
+       HAVE_STRFTIME_L) || ((defined __NetBSD__ || defined __sun) &&
+       REQUIRE_GNUISH_STRFTIME_AM_PM). Do not include hard-locale.h.
+       (libc_hidden_def): On native MS-Windows, use the C locale %c
+       format regardless of language.  I hope that’s good enough.
+       If not, let’s try to think of a way of solving the problem
+       that doesn’t require using hard_locale.
+       * modules/nstrftime (Depends-on): Remove hard-locale.
+       * modules/nstrftime-tests (test_nstrftime_LDADD):
+       Remove $(HARD_LOCALE_LIB).
+       * tests/test-nstrftime.h (locales_test):
+       Add a test that I hope works on French MS-Windows.
+
 2024-06-03  Paul Eggert  <eggert@cs.ucla.edu>
 
        ieee754-h: port to Android NDK r26
index 8eeea45ef4634cb32ad63f1537e94e84586a7ad3..5f1e76833f731ee3eefe6ac087c30532b4f17334 100644 (file)
@@ -92,13 +92,14 @@ extern char *tzname[];
 #include <stdlib.h>
 #include <string.h>
 
-#include <locale.h>
+#if USE_C_LOCALE && HAVE_STRFTIME_L
+# include <locale.h>
+#endif
+
 #if (defined __NetBSD__ || defined __sun) && REQUIRE_GNUISH_STRFTIME_AM_PM
+# include <locale.h>
 # include "localename.h"
 #endif
-#if !USE_C_LOCALE
-# include "hard-locale.h"
-#endif
 
 #include "attribute.h"
 #include <intprops.h>
@@ -1292,10 +1293,7 @@ __strftime_internal (STREAM_OR_CHAR_T *s, STRFTIME_ARG (size_t maxsize)
           subfmt = L_("%a %b %e %H:%M:%S %Y");
 #elif defined _WIN32 && !defined __CYGWIN__
           /* On native Windows, "%c" is "%d/%m/%Y %H:%M:%S" by default.  */
-          if (hard_locale (LC_TIME))
-            subfmt = L_("%a %e %b %Y %H:%M:%S");
-          else
-            subfmt = L_("%a %b %e %H:%M:%S %Y");
+          subfmt = L_("%a %b %e %H:%M:%S %Y");
 #else
           goto underlying_strftime;
 #endif
index a9e86fb1f8d7eeffe71877b3ac7b8908966412ea..e2f852a16b1bb040d82c7e3769f7b2aef23e5fc7 100644 (file)
@@ -14,7 +14,6 @@ c99
 errno
 extensions
 intprops
-hard-locale
 libc-config
 localename-unsafe-limited
 stdbool
index 40ea1bd1995d0045dd586f46d6cb83ed91458114..b5c6ac2fa50a087978510a2effa5c54eb8d7b173 100644 (file)
@@ -26,4 +26,4 @@ TESTS_ENVIRONMENT += \
   LOCALE_FR='@LOCALE_FR@' \
   LOCALE_FR_UTF8='@LOCALE_FR_UTF8@'
 check_PROGRAMS += test-nstrftime
-test_nstrftime_LDADD = $(LDADD) $(SETLOCALE_LIB) $(HARD_LOCALE_LIB)
+test_nstrftime_LDADD = $(LDADD) $(SETLOCALE_LIB)
index b822f3acd17123526bb841f1ff2e91487f980362..80edc7aaecf983c6e6ffa2a8691cab961b3e0f79 100644 (file)
@@ -482,6 +482,7 @@ locales_test (language_t language)
       ASSERT (STREQ (buf, "jeu. 26 oct. 2017 06:40:03") /* glibc, Cygwin */
               || STREQ (buf, "jeu. 26 oct. 06:40:03 2017") /* FreeBSD */
               || STREQ (buf, "Jeu 26 oct 06:40:03 2017") /* macOS, older FreeBSD */
+              || STREQ (buf, "Jeu oct. 26 06:40:03 2017") /* MS-Windows */
               || STREQ (buf, "26 octobre 2017 06:40:03") /* NetBSD */
               || STREQ (buf, "26 octobre 2017 à 06:40:03") /* Solaris (UTF-8) */
               || STREQ (buf, "26 octobre 2017 \340 06:40:03") /* Solaris (ISO-8859-1) */);