Bug 13414 - wrong en_ZA locale decimal point
Summary: wrong en_ZA locale decimal point
Status: RESOLVED INVALID
Alias: None
Product: glibc
Classification: Unclassified
Component: localedata (show other bugs)
Version: 2.14
: P2 normal
Target Milestone: ---
Assignee: GNU C Library Locale Maintainers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-11-16 10:28 UTC by Martin Pitt
Modified: 2014-09-18 13:14 UTC (History)
4 users (show)

See Also:
Host:
Target:
Build:
Last reconfirmed:
fweimer: security-


Attachments
patch (542 bytes, patch)
2011-11-16 10:31 UTC, Martin Pitt
Details | Diff
Patch with updated comments (309 bytes, patch)
2012-01-02 18:44 UTC, jbm
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Martin Pitt 2011-11-16 10:28:21 UTC
Forwarded from https://launchpad.net/bugs/887395:

In the locales package, the file en_ZA (for South Africa) gives a dot '.' as decimal point. However, the Qt library gives a comma ',' as decimal point. This creates problems when a Qt program is trying to communicate with a non Qt program since the decimal point is not the same, numbers are not correctly recognized.

After doing some searching, Qt (see http://developer.qt.nokia.com/wiki/QtLocales) takes its locale information from cldr.unicode.org:
http://unicode.org/cldr/trac/browser/tags/release-2-0-1/posix/en_ZA.UTF-8.src

This page gives a comma as decimal separator. So does Wikipedia (see http://en.wikipedia.org/wiki/Decimal_mark - note that a footnote seems to indicate that this may be a recent official decision).

Having 2 different definitions for the decimal point on the same system can only lead to problems.

-----

http://www.gcis.gov.za/resource_centre/guidelines/styleguide/editorial_styleguide_2011.pdf is an official document which says to use the comma.
Comment 1 Martin Pitt 2011-11-16 10:31:09 UTC
Created attachment 6059 [details]
patch
Comment 2 Ulrich Drepper 2011-12-22 16:26:20 UTC
You have to get the original contributor to agree.
Comment 3 jbm 2012-01-02 18:44:11 UTC
Created attachment 6138 [details]
Patch with updated comments
Comment 4 jbm 2012-01-02 18:45:37 UTC
The patch proposed by martin Pitt is is ok for, but I guess it's better to also fix the comments of the en_ZA file. I uploaded a patch that contains the fix proposed by Martin as well as comment fixes.
Comment 5 Andreas Jaeger 2012-01-03 08:31:51 UTC
Claude, you submitted the files in bugreport 12681. Could you confirm the changes, please?
Comment 6 Claude Paroz 2012-01-03 09:39:58 UTC
I submitted en_ZM, not en_ZA. But I pinged Dwayne as the en_ZA maintainer, and I'm sure he will soon comment.
Comment 7 Martin Pitt 2014-09-18 13:14:25 UTC
I'm withdrawing that, see https://launchpad.net/bugs/1090288