Ticket #6054 (closed defect: fixed)

Bug contains 2 commit(s) | SVN Diffs for #6054

 

Opened 2 years ago

Last modified 2 years ago

The locales japanese.euc and korean.euc on Linux and el.sun_eu_greek on Solaris aren't recognised

Reported by: sean.young@... Assigned to: grhoten
Priority: minor Milestone: 3.8.1
Component: conversion Version: 3.8
Keywords: Cc:
Load: Xref:
Java Version: Operating System: Solaris, Linux
Project (C/J): ICU4C Weeks: 0.1
Review: eric

Description

Testcase depends on #6053

levin:~ $ export LANG=el.sun_eu_greek
levin:~ $ ./uconv --default-code
US-ASCII
levin:~ $ locale charmap
ISO8859-7

The issue is that character set in the locale string isn't be recognised. The attached patch fixes the problem.

Note this issue was found with the shell script (locale.sh):

#!/bin/sh
LANG=$1
export LANG
LIBCNAME=`locale charmap`
ICUNAME=`uconv --default-code`
ICU2NAME=`uconv --list-code $LIBCNAME`
if [ "${ICUNAME}" != "${ICU2NAME}" ];then
        echo $LANG,$LIBCNAME,$ICUNAME,$ICU2NAME
fi

Now execute the following (bash syntax):

for i in $(locale -a); do ./locale.sh $i; done

Attachments

icu-broken-locales.patch (1.1 kB) - added by sean.young@uk.ibm.com on 12/04/07 06:04:54.
Proposed patch

Change History

12/04/07 06:04:54 changed by sean.young@...

  • attachment icu-broken-locales.patch added.

Proposed patch

12/04/07 08:19:10 changed by grhoten

  • xref changed.
  • priority changed from major to minor.
  • owner changed from somebody to grhoten.
  • milestone changed from UNSCH to 3.8.1.
  • keywords deleted.
  • weeks set to 0.1.
  • revw changed.

12/04/07 08:46:20 changed by grhoten

  • revw set to eric.

The test case does not depend on ticket #6053. The patch to putil.c for Solaris was also incorrect, since sun_eu_greek is not ambiguous, and it should be in the normal alias table.

12/05/07 02:28:25 changed by sean.young@...

It looks like all these issues are solved in trunk. I should've checked, please ignore this bugreport.

12/05/07 07:33:39 changed by grhoten

These issues were solved by this bug report, and they were checked into trunk after you submitted this bug report. You can see the changes by looking at the svn diffs on the ticket. This ticket was not ignored.

Trunk is not stable, and it should not be used for production code. You can apply the svn diffs from this ticket to your copy of ICU 3.8.0, or you can wait a few days and use ICU 3.8.1 to get this fix.

12/05/07 12:14:07 changed by eric

  • status changed from new to closed.
  • resolution set to fixed.

Add/Change #6054 (The locales japanese.euc and korean.euc on Linux and el.sun_eu_greek on Solaris aren't recognised)




Anti spam check: