[Bug 420609]

2018-11-07 Thread Digitalfreak
Commit 434d45f fixes the issue.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/420609

Title:
  wrong LC_TIME in Italian

To manage notifications about this bug go to:
https://bugs.launchpad.net/glibc/+bug/420609/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 420609]

2018-11-07 Thread Digitalfreak
Much has changed since the last patch has been attached therefore it
does not apply cleanly now.

For Italian [1] CLDR says that the correct medium date format is "d MMM
y" (5 set 1999), correct medium time format is "HH:mm:ss" (13:25:59),
correct medium date and time format is "d MMM y, HH:mm:ss" (5 set 1999,
13:25:59).

For Italian in Switzerland [2] it provides the same data although marks
them as "unconfirmed".

(In reply to Nicolo' Chieffo from comment #18)
> Ok, so I will change date_fmt to not contain abbreviations.

I've checked some locales and they all contain abbreviations in date_fmt
so I think it is perfectly OK to contain abbreviations.

> What about the timezone? Should it be present for both date_fmt and
d_fmt?

Most (but not all) of the locales contain the timezone in d_t_fmt but
the default C locale does not.  Therefore I think it is correct not to
include the timezone in d_t_fmt but include it in date_fmt.

Now about the patch:

> * d_t_fmt: "%a %e %b %Y, %T, %Z"
> - remove the leading zero in the day of month using '%e' instead of '%b'

Good idea but I think you'd rather use "%-d" which means "do not pad
with zero". "%e" means "pad with space", "%-e" means "do not pad with
space" and therefore has the same effect as "%-d".  Also, I think that
the timezone should be removed.

I will write a new patch and replace it with "%-d" and remove ", %Z".

> - add ',' to separate date, time and timezone

Good, CLDR says that date and time should be separated with a comma.

> * t_fmt: "%H.%M.%S"
> - express the time in the expanded way instead of using '%T', to be sure to
> use the correct values

CLDR says there should be colons rather than dots therefore I will
reject this change.  Also I will restore the format with colons in
date_fmt. "%T" means that "%H:%M:%S" is used literally therefore it is
correct.

> * date_fmt: "%c"
> - make date_fmt and d_t_fmt the same

As said above, date_fmt should contain the timezone while d_t_fmt (which
is used by "%c") should not.  Therefore I think that the correct
date_fmt should be "%a %-d %b %Y, %T, %Z" (ven 14 set 2018, 22:07:41,
CEST)

Also, I will change d_fmt in it_CH from "%d. %m. %y" (which looks bad)
to "%d.%m.%Y" (which is the same as de_CH and is almost the same as CLDR
suggests).  I will not change d_fmt in it_IT because it looks almost the
same as CLDR.  (In both cases CLDR suggests two-digit year while we have
four digits.)

[1] http://st.unicode.org/cldr-apps/v#/it/Gregorian/
[2] http://st.unicode.org/cldr-apps/v#/it_CH/Gregorian/

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/420609

Title:
  wrong LC_TIME in Italian

To manage notifications about this bug go to:
https://bugs.launchpad.net/glibc/+bug/420609/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 420609]

2018-11-07 Thread Digitalfreak
Created attachment 11247
Proposed final solution

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/420609

Title:
  wrong LC_TIME in Italian

To manage notifications about this bug go to:
https://bugs.launchpad.net/glibc/+bug/420609/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 1279558]

2018-09-11 Thread Digitalfreak
1. All links provided in comment 5 consistently confirm that a dot is
used as a grouping (thousands) separator in it_IT, also a comma is used
as a decimal (fraction) separator.  Also CLDR which we usually treat as
an authoritative source confirms the same.  Currently it_IT provides the
same except (as the original reporter noticed) that it does not provide
any grouping separator for LC_NUMERIC.  Therefore I'm going to add a dot
as a grouping separator in LC_NUMERIC in it_IT even if this differs from
the original request.  Note that the same values are currently in
LC_MONETARY in it_IT.

2. But CLDR also confirms that a similar mark as the original reporter
suggests is used in it_CH (Italian language as used in Switzerland).
Currently an ASCII apostrophe is used in glibc but CLDR says it should
be a  character (Right Single Quotation Mark, "’").  It is
correctly transliterated to an ASCII apostrophe while building non-
Unicode locales.  Therefore I'm going to use the  character as a
grouping separator in LC_NUMERIC in it_CH.

3. But all LC_NUMERIC and LC_MONETARY sections in all Swiss locales
(including LC_MONETARY in it_CH) contain just “copy "de_CH"”.  So maybe
LC_NUMERIC should also contain “copy "de_CH"”?  True, de_CH uses an
ASCII apostrophe as a grouping separator while CLDR says it should be
.  Also, CLDR says that a decimal separator for both it_CH and
de_CH should be a dot.  In glibc it is correctly set in all Swiss
locales except it_CH.  Shortly: all Swiss locales should use an
apostrophe (or better the Right Single Quotation Mark) as a grouping
(thousands) separator and a dot as a decimal (fraction) separator.
Therefore instead of what I said in point 2. I am going to use “copy
"de_CH"” in LC_NUMERIC section of it_CH and I'm going to update de_CH to
use  as a thousands separator in both LC_NUMERIC and LC_MONETARY.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1279558

Title:
  Italian (it_IT) locale has empty thousands_sep

To manage notifications about this bug go to:
https://bugs.launchpad.net/glibc/+bug/1279558/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 1279558]

2018-09-11 Thread Digitalfreak
Fixed in master, commit 527f355e5e97bb0f76e88caadb08d2405cacea94.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1279558

Title:
  Italian (it_IT) locale has empty thousands_sep

To manage notifications about this bug go to:
https://bugs.launchpad.net/glibc/+bug/1279558/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs