[issue13441] TestEnUSCollation.test_strxfrm() fails on Solaris

Changes by Jesús Cea Avión : -- nosy: +jcea superseder: -> test_local.TestEnUSCollection failures on Solaris 10 ___ Python tracker ___ __

[issue13441] TestEnUSCollation.test_strxfrm() fails on Solaris

STINNER Victor added the comment: The Solaris buildbot is green, let's close it. I didn't report the bug upstream. Feel free to report it to Oracle! -- resolution: -> fixed status: open -> closed ___ Python tracker

[issue13441] TestEnUSCollation.test_strxfrm() fails on Solaris

STINNER Victor added the comment: I collected the locale list triggering the mbstowcs() bug thanks my previous commit: * hu_HU (ISO8859-2): character U+3020 * de_AT (ISO8859-1): character U+3076 * cs_CZ (ISO8859-2): character U+3020 * sk_SK (ISO8859-2): character U+3020 *

[issue13441] TestEnUSCollation.test_strxfrm() fails on Solaris

Roundup Robot added the comment: New changeset 7ffe3d304487 by Victor Stinner in branch 'default': Issue #13441: Enable the workaround for Solaris locale bug http://hg.python.org/cpython/rev/7ffe3d304487 -- ___ Python tracker

[issue13441] TestEnUSCollation.test_strxfrm() fails on Solaris

Roundup Robot added the comment: New changeset 2a2d0872d993 by Victor Stinner in branch 'default': Issue #13441: Skip some locales (e.g. cs_CZ and hu_HU) on Solaris to workaround http://hg.python.org/cpython/rev/2a2d0872d993 -- ___ Python tracker

[issue13441] TestEnUSCollation.test_strxfrm() fails on Solaris

Roundup Robot added the comment: New changeset 87c6be1e393a by Victor Stinner in branch 'default': Issue #13441: Don't test the hu_HU locale on Solaris to workaround a mbstowcs() http://hg.python.org/cpython/rev/87c6be1e393a -- ___ Python tracker

[issue13441] TestEnUSCollation.test_strxfrm() fails on Solaris

STINNER Victor added the comment: Changeset 489ea02ed351 changed PyUnicode_FromWideChar() and PyUnicode_FromUnicode(): raise a ValueError if a character in not in range [U+; U+10]. test__locale errors: == ERROR: test

[issue13441] TestEnUSCollation.test_strxfrm() fails on Solaris

Roundup Robot added the comment: New changeset 93bab8400ca5 by Victor Stinner in branch 'default': Issue #13441: Log the locale when localeconv() fails http://hg.python.org/cpython/rev/93bab8400ca5 -- ___ Python tracker

[issue13441] TestEnUSCollation.test_strxfrm() fails on Solaris

Stefan Krah added the comment: localeconv_wchar.c runs fine on Ubuntu with hu_HU and fi_FI. I tried on OpenSolaris, but I only have UTF-8 locales. The package with ISO locales seems to be SUNWlang-cs-extra, but Oracle took down http://pkg.opensolaris.org/release/ . --

[issue13441] TestEnUSCollation.test_strxfrm() fails on Solaris

STINNER Victor added the comment: See also the issue #7442. -- ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue13441] TestEnUSCollation.test_strxfrm() fails on Solaris

Changes by Stefan Krah : -- nosy: +skrah ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org

[issue13441] TestEnUSCollation.test_strxfrm() fails on Solaris

STINNER Victor added the comment: localeconv_wchar.c: test program to dump the thousands separator on a locale specified on the command line. I wrote this program to try to reproduce the hu_HU issue, but I cannot reproduce it on OpenIndiana. I only have UTF-8 locales on my OpenIndiana VM, whe

[issue13441] TestEnUSCollation.test_strxfrm() fails on Solaris

Roundup Robot added the comment: New changeset 6f9af4e3c1db by Victor Stinner in branch 'default': Issue #13441: Disable temporary the check on the maximum character until http://hg.python.org/cpython/rev/6f9af4e3c1db -- ___ Python tracker

[issue13441] TestEnUSCollation.test_strxfrm() fails on Solaris

Roundup Robot added the comment: New changeset d6d15fcf5eb6 by Victor Stinner in branch 'default': Issue #13441: Reenable strxfrm() tests on Solaris http://hg.python.org/cpython/rev/d6d15fcf5eb6 -- ___ Python tracker

[issue13441] TestEnUSCollation.test_strxfrm() fails on Solaris

Changes by Antoine Pitrou : -- nosy: -pitrou ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.pytho

[issue13441] TestEnUSCollation.test_strxfrm() fails on Solaris

STINNER Victor added the comment: Here is a more complete output. localeconv() fails in the hu_HU locale for the "thousands_sep" field: localeconv() returns b'\xa0' which is decoded as the wchar_t* string: {U+3020} (len=1). This is an invalid character --- [

[issue13441] TestEnUSCollation.test_strxfrm() fails on Solaris

Roundup Robot added the comment: New changeset acda16de630c by Victor Stinner in branch 'default': Remove temporary hacks for the issue #13441 http://hg.python.org/cpython/rev/acda16de630c -- ___ Python tracker __

[issue13441] TestEnUSCollation.test_strxfrm() fails on Solaris

STINNER Victor added the comment: I dumped some values to try to debug this issue. Last failure in test__locale.test_lc_numeric_basic() on localeconv(): [ 25/361] test_float Decode localeconv() decimal_point: {0x2c} (len=1) Decode localeconv() thousands_sep: {0x2e}

[issue13441] TestEnUSCollation.test_strxfrm() fails on Solaris

Roundup Robot added the comment: New changeset d1b3b1d00811 by Victor Stinner in branch 'default': Another temporary hack to debug the issue #13441 http://hg.python.org/cpython/rev/d1b3b1d00811 -- ___ Python tracker

[issue13441] TestEnUSCollation.test_strxfrm() fails on Solaris

STINNER Victor added the comment: > No, they should be rejected. Allowing them in some specific > places might cause them to leak somewhere else and cause problems, > so I'd rather stick with that range and reject all the chars > >U+10 everywhere. That's why I added a (debug) check to rejec

[issue13441] TestEnUSCollation.test_strxfrm() fails on Solaris

Roundup Robot added the comment: New changeset a19dad38d4e8 by Victor Stinner in branch 'default': Issue #13441: _PyUnicode_CheckConsistency() dumps the string if the maximum http://hg.python.org/cpython/rev/a19dad38d4e8 -- ___ Python tracker

[issue13441] TestEnUSCollation.test_strxfrm() fails on Solaris

Ezio Melotti added the comment: > Is Python supposed to support characters outside U+-U+10 range? No, they should be rejected. Allowing them in some specific places might cause them to leak somewhere else and cause problems, so I'd rather stick with that range and reject all the char

[issue13441] TestEnUSCollation.test_strxfrm() fails on Solaris

Roundup Robot added the comment: New changeset 78123afb3ea4 by Victor Stinner in branch 'default': Issue #13441: Disable temporary localeconv() tests on Solaris http://hg.python.org/cpython/rev/78123afb3ea4 -- ___ Python tracker

[issue13441] TestEnUSCollation.test_strxfrm() fails on Solaris

Changes by STINNER Victor : Removed file: http://bugs.python.org/file23741/strxfrm.c ___ Python tracker ___ ___ Python-bugs-list mailing list

[issue13441] TestEnUSCollation.test_strxfrm() fails on Solaris

STINNER Victor added the comment: > strxfrm(buffer, "a\0", 100) returns 21 (bytes) but only 2 bytes > are written ("\x01\x00"). The next bytes are unchanged. Woops, it was a bug in my program. I attached the fixed version. The correct program writes: strxfrm: len=21 0x01 0x01 0x63 0x01 0x

[issue13441] TestEnUSCollation.test_strxfrm() fails on Solaris

STINNER Victor added the comment: > Is Python supposed to support characters outside U+-U+10 range? If not, PyUnicode_FromUnicode(), PyUnicode_FromWideChar() and PyUnicode_FromKindAndData() should be patched to raise an error if a bigger character is encountered. --

[issue13441] TestEnUSCollation.test_strxfrm() fails on Solaris

Roundup Robot added the comment: New changeset 31baf1363ba1 by Victor Stinner in branch 'default': Issue #13441: Disable temporary strxfrm() tests on Solaris http://hg.python.org/cpython/rev/31baf1363ba1 -- nosy: +python-dev ___ Python tracker

[issue13441] TestEnUSCollation.test_strxfrm() fails on Solaris

New submission from STINNER Victor : I added a test in _PyUnicode_CheckConsistency() (in debug mode) to ensure that all characters of a string are in the range U+-U+10. Locale tests are now failing on Solaris: --- [ 28/361] test__locale Assertion failed: