Bug 13805

Summary: ru_RU: currency should use ',' as radix point
Product: glibc Reporter: Jeffrey Yasskin <jyasskin>
Component: localedataAssignee: Mike FABIAN <maiku.fabian>
Status: RESOLVED FIXED    
Severity: normal CC: libc-locales, maiku.fabian
Priority: P2 Flags: fweimer: security-
Version: 2.13   
Target Milestone: 2.27   
Host: Target:
Build: Last reconfirmed:

Description Jeffrey Yasskin 2012-03-05 10:04:26 UTC
The ru_RU.UTF-8 locale data looks like:
 {decimal_point = 0x7ffff70e4490 ",", thousands_sep = 0x7ffff70e4492 " ",
  grouping = 0x7ffff70e4495 "\003\003", int_curr_symbol = 0x7ffff70e4fb0 "RUB ",
  currency_symbol = 0x7ffff70e4fb5 "руб", mon_decimal_point = 0x7ffff70e4fbc ".",
  mon_thousands_sep = 0x7ffff70e4fbe " , ...

There's a difference between decimal_point and mon_decimal_point that doesn't seem supported in any documentation I've found. ICU uses ',' for both: http://demo.icu-project.org/icu-bin/locexp?d_=en&_=ru_RU.

ICU also includes a '.' at the end of the local currency_symbol, which is supported by wikipedia: http://en.wikipedia.org/wiki/Russian_ruble
Comment 1 Mike FABIAN 2017-08-15 13:09:38 UTC
(In reply to Jeffrey Yasskin from comment #0)
> The ru_RU.UTF-8 locale data looks like:
>  {decimal_point = 0x7ffff70e4490 ",", thousands_sep = 0x7ffff70e4492 " ",
>   grouping = 0x7ffff70e4495 "\003\003", int_curr_symbol = 0x7ffff70e4fb0
> "RUB ",
>   currency_symbol = 0x7ffff70e4fb5 "руб", mon_decimal_point = 0x7ffff70e4fbc
> ".",
>   mon_thousands_sep = 0x7ffff70e4fbe " , ...
> 
> There's a difference between decimal_point and mon_decimal_point that
> doesn't seem supported in any documentation I've found. ICU uses ',' for
> both: http://demo.icu-project.org/icu-bin/locexp?d_=en&_=ru_RU.

Yes, ',' seems correct there.

> ICU also includes a '.' at the end of the local currency_symbol, which is
> supported by wikipedia: http://en.wikipedia.org/wiki/Russian_ruble

I don’t see this, I see only “₽” in that Wikipedia page,
without a dot at the end.
Comment 2 Mike FABIAN 2017-08-29 10:36:29 UTC
(In reply to Mike FABIAN from comment #1)
> (In reply to Jeffrey Yasskin from comment #0)
> > The ru_RU.UTF-8 locale data looks like:
> >  {decimal_point = 0x7ffff70e4490 ",", thousands_sep = 0x7ffff70e4492 " ",
> >   grouping = 0x7ffff70e4495 "\003\003", int_curr_symbol = 0x7ffff70e4fb0
> > "RUB ",
> >   currency_symbol = 0x7ffff70e4fb5 "руб", mon_decimal_point = 0x7ffff70e4fbc
> > ".",
> >   mon_thousands_sep = 0x7ffff70e4fbe " , ...
> > 
> > There's a difference between decimal_point and mon_decimal_point that
> > doesn't seem supported in any documentation I've found. ICU uses ',' for
> > both: http://demo.icu-project.org/icu-bin/locexp?d_=en&_=ru_RU.
> 
> Yes, ',' seems correct there.
> 
> > ICU also includes a '.' at the end of the local currency_symbol, which is
> > supported by wikipedia: http://en.wikipedia.org/wiki/Russian_ruble
> 
> I don’t see this, I see only “₽” in that Wikipedia page,
> without a dot at the end.

And not in CLDR either, ru.xml in CLDR has:

			<currency type="RUB">
				<displayName>российский рубль</displayName>
				<displayName count="one">российский рубль</displayName>
				<displayName count="few">российских рубля</displayName>
				<displayName count="many">российских рублей</displayName>
				<displayName count="other">российского рубля</displayName>
				<symbol>₽</symbol>
				<symbol alt="narrow" draft="contributed">₽</symbol>
			</currency>
Comment 3 Sourceware Commits 2017-08-29 12:02:24 UTC
This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "GNU C Library master sources".

The branch, master has been updated
       via  f11f2f6e145d6fc6b52f6b0733599f8b96595733 (commit)
       via  f8de9568d81bf47bcc85d2918d529b6cd0b19519 (commit)
       via  15cf43a999857de1ad2745b838cb3d73ce45c7e1 (commit)
      from  42496114ec0eb7d6d039d05d4262e109951c600c (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
https://sourceware.org/git/gitweb.cgi?p=glibc.git;h=f11f2f6e145d6fc6b52f6b0733599f8b96595733

commit f11f2f6e145d6fc6b52f6b0733599f8b96595733
Author: Mike FABIAN <mfabian@redhat.com>
Date:   Tue Aug 29 13:39:18 2017 +0200

    Use “,” as mon_decimal_point for ru_RU and os_RU locales (follow CLDR)
    
    	[BZ #13805]
    	* locales/ru_RU (LC_MONETARY): Use “,” for mon_decimal_point
    	(to agree with CLDR).
    	* locales/ru_RU (LC_NUMERIC): Write mon_decimal_point in ASCII
    	for readability.
    	* locales/os_RU (LC_MONETARY): Copy from ru_RU,
    	makes it agree with CLDR.

https://sourceware.org/git/gitweb.cgi?p=glibc.git;h=f8de9568d81bf47bcc85d2918d529b6cd0b19519

commit f8de9568d81bf47bcc85d2918d529b6cd0b19519
Author: Akhilesh Kumar <akhilesh.k@samsung.com>
Date:   Wed Aug 23 09:57:23 2017 +0530

    Add locale for mfe_MU
    
    Add locale for “Morisyen” which is also called “Mauritian Creole”
    and is spoken in Mauritius.
    
    	[BZ #21971]
    	* localedata/SUPPORTED: Add mfe_MU/UTF-8.
    	* localedata/locales/mfe_MU: New File.
    
    	[BZ #21971]
    	* locale/iso-639.def: add Morisyen.

https://sourceware.org/git/gitweb.cgi?p=glibc.git;h=15cf43a999857de1ad2745b838cb3d73ce45c7e1

commit 15cf43a999857de1ad2745b838cb3d73ce45c7e1
Author: Mike FABIAN <mfabian@redhat.com>
Date:   Tue Aug 29 10:54:53 2017 +0200

    Use NFC for the strings in “day”, “yesstr”, and “nostr” in the sgs_LT locale
    
    	* locales/sgs_LT (LC_TIME): Use NFC for the strings in “day”.
    	* locales/sgs_LT (LC_MESSAGES): Use NFC for “yesstr” and “nostr”.

-----------------------------------------------------------------------

Summary of changes:
 ChangeLog                            |    5 +
 locale/iso-639.def                   |    1 +
 localedata/ChangeLog                 |   21 +++++
 localedata/SUPPORTED                 |    1 +
 localedata/locales/{en_SC => mfe_MU} |  136 +++++++++++++++++++++++++---------
 localedata/locales/os_RU             |   16 +----
 localedata/locales/ru_RU             |    4 +-
 localedata/locales/sgs_LT            |    6 +-
 8 files changed, 134 insertions(+), 56 deletions(-)
 copy localedata/locales/{en_SC => mfe_MU} (52%)
Comment 4 Mike FABIAN 2017-08-29 12:04:53 UTC
FIXED (changing mon_decimal_point to "," for both ru_RU and os_RU,
I did not do any change to the currency symbol because neither
Wikipedia nor CLDR seem to have the “.” following the currency
symbol.)