Skip Menu |

This queue is for tickets about the Math-Currency CPAN distribution.

Report information
The Basics
Id: 120394
Status: resolved
Priority: 0/
Queue: Math-Currency
People
Owner: Nobody in particular
Requestors: SREZIC [...] cpan.org
Cc:
AdminCc:
Bug Information
Severity: (no value)
Broken in: 0.50
Fixed in: (no value)

History Show all quoted text
  Sun Feb 26 12:18:37 2017 SREZIC [...] cpan.org - Ticket created
Subject: Currency symbol (RUB) changed in Debian/stretch
My Debian/stretch smoker shows the following failure not seen on other systems: ... # Failed test 'Currency symbol decoded correctly' # at t/009_localeconv_encoding.t line 39. # '₽' # doesn't match '(?^u:руб)' # Looks like you failed 1 test of 2. # Failed test 'ru_RU.UTF-8' # at t/009_localeconv_encoding.t line 44. # Looks like you failed 1 test of 12. t/009_localeconv_encoding.t .. Dubious, test returned 1 (wstat 256, 0x100) Failed 1/12 subtests (less 10 skipped subtests: 1 okay) ... It seems that Debian is now using a dedicated Unicode sign <https://en.wikipedia.org/wiki/Ruble_sign> instead of the cyrillic "rub".
  Sun Feb 26 12:50:52 2017 jpeacock [...] cpan.org - Correspondence added
On Sun Feb 26 12:18:37 2017, SREZIC wrote: Show quoted text
> It seems that Debian is now using a dedicated Unicode sign > <https://en.wikipedia.org/wiki/Ruble_sign> instead of the cyrillic > "rub".
Can you see if it passes with the attached patch? John
Subject: rub.patch
diff -r a6434b67888f t/009_localeconv_encoding.t --- a/t/009_localeconv_encoding.t Wed Jun 22 17:06:59 2016 -0500 +++ b/t/009_localeconv_encoding.t Sun Feb 26 12:50:02 2017 -0500 @@ -19,9 +19,9 @@ 'en_GB' => '£', 'en_GB.UTF-8' => '£', 'en_GB.ISO8859-1' => '£', - 'ru_RU' => qr/руб/, # on some systems, this is 'руб', on others is 'руб.' - 'ru_RU.UTF-8' => qr/руб/, - 'ru_RU.KOI8-R' => qr/руб/, + 'ru_RU' => qr/(руб|₽)/, # on some systems, this is 'руб', on others is 'руб.' + 'ru_RU.UTF-8' => qr/(руб|₽)/, + 'ru_RU.KOI8-R' => qr/(руб|₽)/, 'zh_CN.GB2312' => '￥', 'zh_CN' => '￥', 'zh_CN.GB2312' => '￥',
  Sun Feb 26 12:50:53 2017 The RT System itself - Status changed from 'new' to 'open'
  Sun Feb 26 13:15:33 2017 SREZIC [...] cpan.org - Correspondence added
On 2017-02-26 12:50:52, JPEACOCK wrote: Show quoted text
> On Sun Feb 26 12:18:37 2017, SREZIC wrote:
> > It seems that Debian is now using a dedicated Unicode sign > > <https://en.wikipedia.org/wiki/Ruble_sign> instead of the cyrillic > > "rub".
> > Can you see if it passes with the attached patch?
Yes, the problem is gone with this patch.
  Sun Feb 26 13:56:30 2017 jpeacock [...] cpan.org - Correspondence added
On Sun Feb 26 13:15:33 2017, SREZIC wrote: Show quoted text
> On 2017-02-26 12:50:52, JPEACOCK wrote:
> > On Sun Feb 26 12:18:37 2017, SREZIC wrote:
> > > It seems that Debian is now using a dedicated Unicode sign > > > <https://en.wikipedia.org/wiki/Ruble_sign> instead of the cyrillic > > > "rub".
> > > > Can you see if it passes with the attached patch?
> > Yes, the problem is gone with this patch.
Cool. This module is now maintained by Michael Schout <mschout@gkg.net>. I'll make sure he gets the patch.
  Sun Feb 26 13:56:41 2017 jpeacock [...] cpan.org - Status changed from 'open' to 'patched'
  Sun Feb 26 14:48:23 2017 jpeacock [...] cpan.org - Correspondence added
https://github.com/mschout/perl-math-currency/pull/1
  Sun Feb 26 14:48:29 2017 jpeacock [...] cpan.org - Status changed from 'patched' to 'resolved'
  Sun Feb 26 18:22:05 2017 MSCHOUT [...] cpan.org - Correspondence added
Thanks I'll get a release out tomorrow -- Regards, Michael Schout
  Mon Feb 27 10:53:34 2017 MSCHOUT [...] cpan.org - Correspondence added
v0.51 has been uploaded to CPAN which fixes this issue. -- Regards, Michael Schout