tr_TR has wrong currency

Bug #34738 reported by Erdal Ronahi
8
Affects Status Importance Assigned to Milestone
belocs-locales-bin (Ubuntu)
Fix Released
Medium
Martin Pitt
belocs-locales-data (Debian)
New
Undecided
Denis Barbier
iso-codes (Ubuntu)
Fix Released
Medium
Martin Pitt
langpack-locales (Ubuntu)
Fix Released
Medium
Martin Pitt

Bug Description

there is an error in the tr_TR locale definition (/usr/share/i18n/locales

In the section LC_MONETARY there is the following line:

int_curr_symbol "<U0054><U0052><U0059><U0020>"

The space character at the end is producing errors, because the system wants only three characters.

Revision history for this message
Erdal Ronahi (erdalronahi) wrote :

In Dapper, this causes the command

sudo dpkg-reconfigure locales

to hang on my computer. It gives an incomplete error message (like 'int_curr_symbol' is ou) and then stops. I am using the locale ku_TR, which copies the currency definition from tr_TR.

Martin Pitt (pitti)
Changed in langpack-locales:
assignee: nobody → pitti
status: Unconfirmed → In Progress
Revision history for this message
Martin Pitt (pitti) wrote :

Denis, I subscribed you since this should be fixed in Debian, too. Are you fine with these Launchpad emails? It's easier than filing a separate Debian bug report for each issue.

Changed in belocs-locales-data:
assignee: nobody → barbier-linuxfr
Revision history for this message
Martin Pitt (pitti) wrote :

 langpack-locales (2.3.12) dapper; urgency=low
 .
   * locales/en_NZ: Add AM/PM definition (Malone #34262)
   * locales/tr_TR: Fix stray space in int_curr_symbol (Malone #34738)
   * locales/eo: Rename to eo_XX, since that's the format that other programs
     expect (Malone #23435)
   * locales/es_CO: Change standard paper size from A4 to US Letter. (Malone
     #27622)

Changed in langpack-locales:
status: In Progress → Fix Released
Revision history for this message
Denis Barbier (barbier) wrote : Re: [Bug 34738] tr_TR has wrong currency

On Mon, Mar 20, 2006 at 03:50:55PM -0000, Martin Pitt wrote:
> Public bug report changed:
> https://launchpad.net/malone/bugs/34738
>
> Task: ubuntu langpack-locales
> Status: In Progress => Fix Released
>
> Denis, I subscribed you since this should be fixed in Debian, too. Are
> you fine with these Launchpad emails? It's easier than filing a separate
> Debian bug report for each issue.

No problem, but I am getting sick of web services and do not want to
request a login and authenticate. Thus if this mail does not reach
your system, I will do nothing, it is much easier this way for me ;)

> Comment:
> langpack-locales (2.3.12) dapper; urgency=low
> .
> * locales/en_NZ: Add AM/PM definition (Malone #34262)

Will do, a bug report has already been filed in the BTS.

> * locales/tr_TR: Fix stray space in int_curr_symbol (Malone #34738)

This bug report is wrong.

> * locales/eo: Rename to eo_XX, since that's the format that other programs
> expect (Malone #23435)

I will keep eo.

> * locales/es_CO: Change standard paper size from A4 to US Letter. (Malone
> #27622)

Already in Debian.

Denis

Revision history for this message
Erdal Ronahi (erdalronahi) wrote :

> > * locales/tr_TR: Fix stray space in int_curr_symbol (Malone #34738)
>
> This bug report is wrong.

What exactly is wrong?

Revision history for this message
Erdal Ronahi (erdalronahi) wrote :

In fact, the locale definition file is correct in the latest Dapper. But I still get the same error!

Revision history for this message
Martin Pitt (pitti) wrote :

Hi Denis,

Denis Barbier [2006-03-20 21:25 +0100]:
> No problem, but I am getting sick of web services and do not want to
> request a login and authenticate. Thus if this mail does not reach
> your system, I will do nothing, it is much easier this way for me ;)

That's fine, you are already known to our bug tracker, and using it
like Debian's BTS (purely with email) works fine.

> > * locales/tr_TR: Fix stray space in int_curr_symbol (Malone #34738)
>
> This bug report is wrong.

Ok, thanks, will sort that out with Erdal then.

> > * locales/eo: Rename to eo_XX, since that's the format that other programs
> > expect (Malone #23435)
>
> I will keep eo.

So you fixed the locale name in /usr/share/i18n/SUPPORTED instead? If
the rather nonsensical _XX can be avoided, I'd prefer to do that, too.

Nice.

Thank you,

Martin
--
Martin Pitt http://www.piware.de
Ubuntu Developer http://www.ubuntu.com
Debian Developer http://www.debian.org

In a world without walls and fences, who needs Windows and Gates?

Revision history for this message
Martin Pitt (pitti) wrote :

Reopening

Changed in langpack-locales:
status: Fix Released → Confirmed
Revision history for this message
Martin Pitt (pitti) wrote :

So the space is correct, localedef expects four characters. However, with the original sequence, localedef complains with

  LC_MONETARY: value of field `int_curr_symbol' does not correspond to a valid name in ISO 4217

And indeed there is no 'TRY' defined in /usr/share/i18n/locales/iso-4217.def, just

  DEFINE_INT_CURR("TRL") /* Turkish Lira */

So is "TRL" the correct Turkish currency?

Changed in langpack-locales:
status: Confirmed → Needs Info
Revision history for this message
Erdal Ronahi (erdalronahi) wrote :

No, the currency in Turkey has changed in 2005. It was "Turkish Lira" (TRL), now it is "New Turkish Lira" (TRY). 1 TRY equals 1.000.000 TRL.

You can see this on http://www.iso.org/iso/en/prods-services/popstds/currencycodeslist.html#SZ :
TURKEY New Turkish Lira TRY 949

So maybe the iso-4217 list in Ubuntu is out of date.

Revision history for this message
Denis Barbier (barbier) wrote :

On Tue, Mar 21, 2006 at 07:48:05AM -0000, Martin Pitt wrote:
> Denis Barbier [2006-03-20 21:25 +0100]:
> > No problem, but I am getting sick of web services and do not want to
> > request a login and authenticate. Thus if this mail does not reach
> > your system, I will do nothing, it is much easier this way for me ;)
>
> That's fine, you are already known to our bug tracker, and using it
> like Debian's BTS (purely with email) works fine.

Cool.

> > > * locales/tr_TR: Fix stray space in int_curr_symbol (Malone #34738)
> >
> > This bug report is wrong.
>
> Ok, thanks, will sort that out with Erdal then.

As said by Erdal, Turkish currency is TRY, localedef needs to be
updated. But I do not know why locales generation hangs, this
is not normal.

> > > * locales/eo: Rename to eo_XX, since that's the format that other programs
> > > expect (Malone #23435)
> >
> > I will keep eo.
>
> So you fixed the locale name in /usr/share/i18n/SUPPORTED instead?

Right. IIRC I initially chose eo_XX to be compatible with Mandrake.
As explained in the changelog, I met Goto Masanori at debconf 5
and he told me that he prefers eo. I had no strong feeling about
this and switched to eo.

> If the rather nonsensical _XX can be avoided, I'd prefer to do that,
> too.

FWIW, ISO 3166 specifies that AA, QM-QZ and XA-XZ codes are reserved
for private use, this is why eo_XX may have a justification, at least
IMHO ;)

Denis

Revision history for this message
Erdal Ronahi (erdalronahi) wrote :

>
> As said by Erdal, Turkish currency is TRY, localedef needs to be
> updated. But I do not know why locales generation hangs, this
> is not normal.
>

After the last update, I have no more serious problems. Maybe the hanging
was not because of that, it's gone anyway.

Erdal

Martin Pitt (pitti)
Changed in langpack-locales:
status: Needs Info → Confirmed
Revision history for this message
Martin Pitt (pitti) wrote :

So, the currency needs to be updated in iso-codes, adding task.

Changed in iso-codes:
assignee: nobody → pitti
status: Unconfirmed → In Progress
Revision history for this message
Martin Pitt (pitti) wrote :

Package ready, will upload after flight-6 freeze.

Changed in iso-codes:
status: In Progress → Fix Committed
Revision history for this message
Martin Pitt (pitti) wrote :

Fixed package prepared, will upload after flight-6 freeze.

Changed in belocs-locales-bin:
assignee: nobody → pitti
status: Unconfirmed → Fix Committed
Revision history for this message
Martin Pitt (pitti) wrote :

Fixed langpack-locales package prepared (re-added the space); with the fixed belocs-locale-bin package, tr_TR[.UTF-8] now works like charm. :)

Changed in langpack-locales:
status: Confirmed → Fix Committed
Revision history for this message
Martin Pitt (pitti) wrote :

 belocs-locales-bin (2.3.5-5ubuntu4) dapper; urgency=low
 .
   * locale/iso-4217.def: Add "New Turkish Lira" (TRY). Closes: LP#34738
   * sbin/locale-gen: Do not completely fail if a single locale cannot be
     generated; just print 'failed' and continue. Closes: LP#33123.

Changed in belocs-locales-bin:
status: Fix Committed → Fix Released
Revision history for this message
Martin Pitt (pitti) wrote :

 iso-codes (0.49-1ubuntu2) dapper; urgency=low
 .
   * iso_4217/iso_4217.xml: Add "New Turkish Lira" (TRY, ISO-Code 949).
     Closes: LP#34738

Changed in iso-codes:
status: Fix Committed → Fix Released
Revision history for this message
Martin Pitt (pitti) wrote :

 langpack-locales (2.3.13) dapper; urgency=low
 .
   * locales/tr_TR: Re-add trailing space in int_curr_symbol, it is correct.
     Closes: LP#34738
   * Fix Catalan locales (sync with Debian), thanks to Robert Millan.
     - Add Euro support to ca_FR and ca_AD.
     - Remove obsolete ca_FR@euro and ca_AD@euro.
     - Add ca_IT.
     - List them all in SUPPORTED file.
     - Closes: LP#37011

Changed in langpack-locales:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.