Bug #41285 for Filter: Tests may fail if non-english locale is used

Fri Nov 28 02:55:16 2008 SREZIC [...] cpan.org - Ticket created

Subject:

Tests may fail if non-english locale is used

With my default locale settings (de_DE.ISO8859-1) the exec and sh tests fail. If I set the locale to "C", then everything passes. Feel free to ask if you need more diagnostics. Regards, Slaven

Thu Jan 24 15:43:09 2013 SREZIC [...] cpan.org - Correspondence added

On 2008-11-28 02:55:16, SREZIC wrote: Show quoted text

> With my default locale settings (de_DE.ISO8859-1) the exec and sh tests > fail. If I set the locale to "C", then everything passes. > > Feel free to ask if you need more diagnostics. >

This is still failing. In the case of the "sh" test, it's a non-portable usage of tr. If a German locale is active, then the output is as follows: $ echo t | tr '[a-z]' '[A-Z]' U With a "C" locale the output is as expected: $ echo t | env LANG=C LC_ALL=C tr '[a-z]' '[A-Z]' T To be portable, a character class should be used: $ echo t | tr '[:lower:]' '[:upper:]' T Regards, Slaven

Thu Jan 24 15:43:11 2013 SREZIC [...] cpan.org - Status changed from 'new' to 'open'

Thu Mar 28 15:26:44 2013 RURBAN [...] cpan.org - Taken

Fri Mar 29 10:37:52 2013 RURBAN [...] cpan.org - Correspondence added

On Thu Jan 24 15:43:09 2013, SREZIC wrote: Show quoted text

> On 2008-11-28 02:55:16, SREZIC wrote:

> > With my default locale settings (de_DE.ISO8859-1) the exec and sh tests > > fail. If I set the locale to "C", then everything passes. > > Feel free to ask if you need more diagnostics. > >

> > This is still failing.

Hi Slaven, I don't have any non-english systems anymore around. Can you try my attempt if I fixed it? https://github.com/rurban/Filter/commits/ Thanks -- Reini Urban

Fri Mar 29 12:58:21 2013 RURBAN [...] cpan.org - Correspondence added

Please test 1.46 -- Reini Urban

Fri Mar 29 12:58:22 2013 RURBAN [...] cpan.org - Status changed from 'open' to 'resolved'

Tue Apr 02 16:27:43 2013 SREZIC [...] cpan.org - Correspondence added

On 2013-03-29 12:58:21, RURBAN wrote: Show quoted text

> Please test 1.46

1.48 still fails on my system, but see https://github.com/rurban/Filter/pull/1 Adding some explanations for this pull request: - On my system is only LC_ALL set. But the test scripts used only to check for the LANG env var, so the code setting the env vars to C was never triggered. - But checking LC_ALL and LANG is not enough, actually LC_CTYPE needs to be checked. - Instead of checking all possible env vars, it's probably better to use POSIX::setlocale (like I am proposing in the pull request) - The code setting the env vars should also set LC_CTYPE=C - And the BEGIN{...} block was missing here. With the changes the tests pass on my FreeBSD system with a German and a C locale. Things still work on a Debian/wheezy system, both with a German and a C locale. Regards, Slaven

Tue Apr 02 16:27:44 2013 The RT System itself - Status changed from 'resolved' to 'open'

Tue Apr 02 19:45:42 2013 RURBAN [...] cpan.org - Correspondence added

Thanks for the fix. Published as 1.49 -- Reini Urban

Tue Apr 02 19:45:43 2013 RURBAN [...] cpan.org - Status changed from 'open' to 'resolved'