Eli Zaretskii wrote:
My impression was that REPLACE_NL_LANGINFO cannot be defined on
MS-Windows, since there's no nl_langinfo on Windows to replace.
OK, in that case it doesn't matter whether that part of the change is
applied, so the currently installed code should be OK. Thanks.
> Date: Mon, 07 Jul 2014 15:23:28 -0700
> From: Paul Eggert
> CC: bug-gnulib@gnu.org
>
> Eli Zaretskii wrote:
> > Thanks. I needed the follow-up patch below to be able to build Guile
> > and run all the tests successfully. I hope the comments within the
> > patch explain why those changes are n
Eli Zaretskii wrote:
Thanks. I needed the follow-up patch below to be able to build Guile
and run all the tests successfully. I hope the comments within the
patch explain why those changes are needed on MS-Windows.
OK, thanks, but shouldn't those improvements also apply to nl_langinfo
(CODES
> Date: Sat, 05 Jul 2014 14:59:05 -0700
> From: Paul Eggert
> CC: bug-gnulib@gnu.org
>
> Revised proposed patch attached.
Thanks. I needed the follow-up patch below to be able to build Guile
and run all the tests successfully. I hope the comments within the
patch explain why those changes are
Thanks, I tried that out on Fedora 20 (pretending that its nl_langinfo
didn't exist) and had some trouble building and testing it, indicating
trouble for other POSIXish platforms. I fixed the problems I found. A
few things:
* nl_langinfo (FRAC_DIGITS) returns a pointer to a character, not a
Sorry, please use the patch below instead of the one I sent a few
minutes ago.
diff --git a/ChangeLog b/ChangeLog
index eccf03e..8a1a9c8 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,24 @@
+2014-07-05 Eli Zaretskii
+
+ nl_langinfo: CODESET on MS-Windows and more items from localeconv.
> Date: Sat, 05 Jul 2014 10:12:29 -0700
> From: Paul Eggert
>
> Eli Zaretskii wrote:
> > Ping! Could someone please respond, and either accept the patch or
> > explain why it cannot be accepted?
>
> Since nobody has commented and it works for you I'm inclined to apply
> it, unless Daiki speaks
Eli Zaretskii wrote:
Ping! Could someone please respond, and either accept the patch or
explain why it cannot be accepted?
Since nobody has commented and it works for you I'm inclined to apply
it, unless Daiki speaks up. Could you write a proper ChangeLog entry
for it please? Typical Gnuli
> From: l...@gnu.org (Ludovic Courtès)
> Cc: Eli Zaretskii
> Date: Thu, 03 Jul 2014 10:16:41 +0200
>
> >> --- lib/nl_langinfo.c~02014-02-15 01:00:33 +0200
> >> +++ lib/nl_langinfo.c 2014-06-11 17:50:54 +0300
> >
> > Ping! Could someone please respond, and either accept the patch or
> >
Eli Zaretskii skribis:
>> Date: Wed, 11 Jun 2014 18:38:09 +0300
>> From: Eli Zaretskii
>>
>> > Date: Tue, 10 Jun 2014 21:08:08 +0300
>> > From: Eli Zaretskii
>> >
>> > Currently, gnulib's nl_langinfo provides on C-locale names for week
>> > days and months, thus any program that uses it can o
> Date: Wed, 11 Jun 2014 18:38:09 +0300
> From: Eli Zaretskii
>
> > Date: Tue, 10 Jun 2014 21:08:08 +0300
> > From: Eli Zaretskii
> >
> > Currently, gnulib's nl_langinfo provides on C-locale names for week
> > days and months, thus any program that uses it can only speak English
> > in these co
> Date: Tue, 10 Jun 2014 21:08:08 +0300
> From: Eli Zaretskii
>
> Currently, gnulib's nl_langinfo provides on C-locale names for week
> days and months, thus any program that uses it can only speak English
> in these contexts.
>
> How about the following patch, which uses strftime to produce
> l
Currently, gnulib's nl_langinfo provides on C-locale names for week
days and months, thus any program that uses it can only speak English
in these contexts.
How about the following patch, which uses strftime to produce
localized names for these?
--- lib/nl_langinfo.c~0 2014-02-15 01:00:33 +0200
+
13 matches
Mail list logo
