unofficial mirror of libc-alpha@sourceware.org
 help / color / mirror / Atom feed
* [PATCH] sl_SI locale: Use "." as the thousands separator (bug 25233)
@ 2020-01-03 11:23 Rafal Luzynski
  2020-01-06 15:07 ` Siddhesh Poyarekar
  0 siblings, 1 reply; 3+ messages in thread
From: Rafal Luzynski @ 2020-01-03 11:23 UTC (permalink / raw)
  To: Siddhesh Poyarekar, libc-alpha; +Cc: Florian Weimer, Jure Sah

The patch is trivial but as we are in the (slushy) freeze period
the question is: Is it OK to commit today/tonight?  Siddhesh?

Regards,

Rafal

---------------------------8<---------------------------

From: =?UTF-8?q?Rafa=C5=82=20Lu=C5=BCy=C5=84ski?=
 <digitalfreak@lingonborough.com>
Date: Fri, 3 Jan 2020 12:11:20 +0100

This is correct according to CLDR [1] and Florian Weimer's quick
research. [2]

[1] https://st.unicode.org/cldr-apps/v#/sl/Symbols/
[2] https://sourceware.org/bugzilla/show_bug.cgi?id=25233#c0
---
 localedata/locales/sl_SI | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/localedata/locales/sl_SI b/localedata/locales/sl_SI
index 8e290175ba..3c15937801 100644
--- a/localedata/locales/sl_SI
+++ b/localedata/locales/sl_SI
@@ -2134,7 +2134,7 @@ LC_MONETARY
 int_curr_symbol           "EUR "
 currency_symbol           "<U20AC>"
 mon_decimal_point         ","
-mon_thousands_sep         "<U202F>"
+mon_thousands_sep         "."
 mon_grouping              3;3
 positive_sign             ""
 negative_sign             "-"
@@ -2150,7 +2150,7 @@ END LC_MONETARY
 
 LC_NUMERIC
 decimal_point             ","
-thousands_sep             "<U202F>"
+thousands_sep             "."
 grouping                  0;0
 END LC_NUMERIC
 
-- 
2.21.0

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] sl_SI locale: Use "." as the thousands separator (bug 25233)
  2020-01-03 11:23 [PATCH] sl_SI locale: Use "." as the thousands separator (bug 25233) Rafal Luzynski
@ 2020-01-06 15:07 ` Siddhesh Poyarekar
  2020-01-07 23:20   ` Rafal Luzynski
  0 siblings, 1 reply; 3+ messages in thread
From: Siddhesh Poyarekar @ 2020-01-06 15:07 UTC (permalink / raw)
  To: Rafal Luzynski, libc-alpha; +Cc: Florian Weimer, Jure Sah

On 03/01/20 4:53 pm, Rafal Luzynski wrote:
> The patch is trivial but as we are in the (slushy) freeze period
> the question is: Is it OK to commit today/tonight?  Siddhesh?
> 
> Regards,
> 
> Rafal
> 

This is fine.

Reviewed-by: Siddhesh Poyarekar <siddhesh@sourceware.org>

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] sl_SI locale: Use "." as the thousands separator (bug 25233)
  2020-01-06 15:07 ` Siddhesh Poyarekar
@ 2020-01-07 23:20   ` Rafal Luzynski
  0 siblings, 0 replies; 3+ messages in thread
From: Rafal Luzynski @ 2020-01-07 23:20 UTC (permalink / raw)
  To: Siddhesh Poyarekar, libc-alpha; +Cc: Florian Weimer, Jure Sah

6.01.2020 16:07 Siddhesh Poyarekar <siddhesh@gotplt.org> wrote:
> [...]
> This is fine.
> 
> Reviewed-by: Siddhesh Poyarekar <siddhesh@sourceware.org>

Thank you, it's been pushed now.

Regards,

Rafal

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2020-01-07 23:22 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-01-03 11:23 [PATCH] sl_SI locale: Use "." as the thousands separator (bug 25233) Rafal Luzynski
2020-01-06 15:07 ` Siddhesh Poyarekar
2020-01-07 23:20   ` Rafal Luzynski

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).