[Bug localedata/23724] New: Albanian date formats are incorrect

public inbox for libc-locales@sourceware.org
 help / color / mirror / Atom feed

* [Bug localedata/23724] New: Albanian date formats are incorrect
@ 2018-09-28 20:54 digitalfreak at lingonborough dot com
  2018-09-28 21:18 ` [Bug localedata/23724] " fweimer at redhat dot com
                   ` (3 more replies)
  0 siblings, 4 replies; 5+ messages in thread
From: digitalfreak at lingonborough dot com @ 2018-09-28 20:54 UTC (permalink / raw)
  To: libc-locales

https://sourceware.org/bugzilla/show_bug.cgi?id=23724

            Bug ID: 23724
           Summary: Albanian date formats are incorrect
           Product: glibc
           Version: 2.28
            Status: NEW
          Severity: normal
          Priority: P2
         Component: localedata
          Assignee: unassigned at sourceware dot org
          Reporter: digitalfreak at lingonborough dot com
                CC: libc-locales at sourceware dot org
  Target Milestone: ---

Albanian locale data (sq_AL) in glibc uses the date format "%Y-%b-%d" which is
kinda weird because it renders something like "2018-Sht-28".  According to CLDR
[1] it should be "28 Sht 2018" for dates having an abbreviated month name and
"28.9.18" for short (numerical only) dates.

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

-- 
You are receiving this mail because:
You are on the CC list for the bug.

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [Bug localedata/23724] Albanian date formats are incorrect
  2018-09-28 20:54 [Bug localedata/23724] New: Albanian date formats are incorrect digitalfreak at lingonborough dot com
@ 2018-09-28 21:18 ` fweimer at redhat dot com
  2018-10-01 10:38 ` digitalfreak at lingonborough dot com
                   ` (2 subsequent siblings)
  3 siblings, 0 replies; 5+ messages in thread
From: fweimer at redhat dot com @ 2018-09-28 21:18 UTC (permalink / raw)
  To: libc-locales

https://sourceware.org/bugzilla/show_bug.cgi?id=23724

Florian Weimer <fweimer at redhat dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |fweimer at redhat dot com
              Flags|                            |security-

--- Comment #1 from Florian Weimer <fweimer at redhat dot com> ---
Do we really want to use %d.m.%y for the short date format?  It's not Y2K-safe,
after all.

<https://www.kryeministria.al/newsroom/vendime-te-miratuara-ne-mbledhjen-e-keshillit-te-ministrave-date-31-korrik-2018/>
uses %d.%m.%Y.

-- 
You are receiving this mail because:
You are on the CC list for the bug.

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [Bug localedata/23724] Albanian date formats are incorrect
  2018-09-28 20:54 [Bug localedata/23724] New: Albanian date formats are incorrect digitalfreak at lingonborough dot com
  2018-09-28 21:18 ` [Bug localedata/23724] " fweimer at redhat dot com
@ 2018-10-01 10:38 ` digitalfreak at lingonborough dot com
  2018-12-28 21:18 ` cvs-commit at gcc dot gnu.org
  2018-12-28 21:23 ` digitalfreak at lingonborough dot com
  3 siblings, 0 replies; 5+ messages in thread
From: digitalfreak at lingonborough dot com @ 2018-10-01 10:38 UTC (permalink / raw)
  To: libc-locales

https://sourceware.org/bugzilla/show_bug.cgi?id=23724

--- Comment #2 from Rafal Luzynski <digitalfreak at lingonborough dot com> ---
I think it is not a problem because it is only how the dates are displayed,
also it does not mean that using 4 digits is impossible.  Y2K problem, as far
as I understand it, is about how dates are stored internally.  If some locale
conventions want to display year using 2 digits I think it is OK.  A good
argument for me is when CLDR says so.  Your link shows that 4 digits are also
used in Albanian language, it does not mean that 2 digit year is prohibited.

A quick grep shows that there are currently 106 locales using "%y" in d_fmt
including POSIX and the default C locale (a fallback when setting a locale
fails) plus 8 locales using "%Oy" or "%Ey", compared to the total of 294
locales which actually provide d_fmt (rather than using "copy").  I don't mind
if we set a rule which says that "%y" should never be suggested by glibc by
default and "%Y" should be used instead but we would have to change POSIX and C
locales as well which I think is out of our scope.

-- 
You are receiving this mail because:
You are on the CC list for the bug.

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [Bug localedata/23724] Albanian date formats are incorrect
  2018-09-28 20:54 [Bug localedata/23724] New: Albanian date formats are incorrect digitalfreak at lingonborough dot com
  2018-09-28 21:18 ` [Bug localedata/23724] " fweimer at redhat dot com
  2018-10-01 10:38 ` digitalfreak at lingonborough dot com
@ 2018-12-28 21:18 ` cvs-commit at gcc dot gnu.org
  2018-12-28 21:23 ` digitalfreak at lingonborough dot com
  3 siblings, 0 replies; 5+ messages in thread
From: cvs-commit at gcc dot gnu.org @ 2018-12-28 21:18 UTC (permalink / raw)
  To: libc-locales

https://sourceware.org/bugzilla/show_bug.cgi?id=23724

--- Comment #3 from cvs-commit at gcc dot gnu.org <cvs-commit at gcc dot gnu.org> ---
This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "GNU C Library master sources".

The branch, master has been updated
       via  989182c40a54c3022ff75c3c313479231dd36602 (commit)
       via  27841a7d5abe36c48b41b33d39ca8e5e14b9d36f (commit)
      from  5fc3eca750791ed7e389026a70864687b4bf8379 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
https://sourceware.org/git/gitweb.cgi?p=glibc.git;h=989182c40a54c3022ff75c3c313479231dd36602

commit 989182c40a54c3022ff75c3c313479231dd36602
Author: Rafal Luzynski <digitalfreak@lingonborough.com>
Date:   Tue Oct 9 01:16:36 2018 +0200

    Multiple locales: Use the correct 12-hour time formats (bug 10496).

    It has been discovered that some locales use the 12-hour time formats but
    do not use any AM/PM indicator thus making the time ambiguous.  This
    commit adds "%p" wherever it was missing.  In some cases it has been
    identified that a locale should use 24-hour time format rather than
    12-hour.  All time formats come from CLDR but this commit introduces as
    few changes as possible (for example, it tries not to change the time zone
    display).  For the locales which are not supported by CLDR the consistency
    with similar locales (which means the same language or the same country)
    has been preserved: if the time formats were the same before the change
    then they are still the same after the change.

    The time format updates can be roughly summarized as follows:

    * Most of the locales of Djibouti, Eritrea, and Ethiopia now use
    "%l:%M:%S %p".
    * Most of the locales of India and some surrounding countries (Bangladesh,
    Nepal etc.) now use "%I:%M:%S %p %Z".
    * Most of the Arabic locales now use "%Z %I:%M:%S %p".
    * Ge'ez language (Eritrea and Ethiopia) now uses "%l:%M:%S፡%p" (note the
    consistent use of Ethiopic wordspace character).
    * Tamil (India) now uses "%p %I:%M:%S %Z".
    * Chinese (Hong Kong) t_fmt now uses "%p %I<U6642>%M<U5206>%S<U79D2> %Z".
    * Additionally, the following locales have been switched from 12-hour time
    formats to 24-hour, according to CLDR: Arabic (Morocco), Maltese, Somali
    (Kenya), and Tamil (Sri Lanka).
    * Finally, the Bulgarian, Czech, and Slovak locales used 24-hour time
    format correctly but their t_fmt_ampm field was not empty containing
    12-hour time format which was incorrect so it is now replaced with an
    empty string.

        [BZ #10496]
        * localedata/locales/aa_DJ (t_fmt): Set to "%l:%M:%S %p".
        (t_fmt_ampm): Likewise.
        * localedata/locales/aa_ER (t_fmt): Likewise.
        (t_fmt_ampm): Likewise.
        * localedata/locales/aa_ER@saaho (t_fmt): Likewise.
        (t_fmt_ampm): Likewise.
        * localedata/locales/aa_ET (t_fmt): Likewise.
        (t_fmt_ampm): Likewise.
        * localedata/locales/am_ET (t_fmt): Likewise.
        (t_fmt_ampm): Likewise.
        * localedata/locales/byn_ER (t_fmt): Likewise.
        (t_fmt_ampm): Likewise.
        * localedata/locales/om_ET (t_fmt): Likewise.
        (t_fmt_ampm): Likewise.
        * localedata/locales/sid_ET (t_fmt): Likewise.
        (t_fmt_ampm): Likewise.
        * localedata/locales/so_DJ (t_fmt): Likewise.
        (t_fmt_ampm): Likewise.
        * localedata/locales/so_ET (t_fmt): Likewise.
        (t_fmt_ampm): Likewise.
        * localedata/locales/so_SO (t_fmt): Likewise.
        (t_fmt_ampm): Likewise.
        * localedata/locales/ti_ER (t_fmt): Likewise.
        (t_fmt_ampm): Likewise.
        * localedata/locales/ti_ET (t_fmt): Likewise.
        (t_fmt_ampm): Likewise.
        * localedata/locales/tig_ER (t_fmt): Likewise.
        (t_fmt_ampm): Likewise.
        * localedata/locales/wal_ET (t_fmt): Likewise.
        (t_fmt_ampm): Likewise.

        * localedata/locales/anp_IN (t_fmt): Set to "%I:%M:%S %p %Z".
        * localedata/locales/ar_IN (t_fmt): Likewise.
        * localedata/locales/bhb_IN (t_fmt): Likewise.
        * localedata/locales/bho_IN (t_fmt): Likewise.
        * localedata/locales/bi_VU (t_fmt): Likewise.
        * localedata/locales/bn_BD (t_fmt): Likewise.
        * localedata/locales/bn_IN (t_fmt): Likewise.
        * localedata/locales/brx_IN (t_fmt): Likewise.
        * localedata/locales/doi_IN (t_fmt): Likewise.
        * localedata/locales/en_HK (t_fmt): Likewise.
        (t_fmt_ampm): Likewise.
        * localedata/locales/en_IN (t_fmt): Likewise.
        * localedata/locales/en_PH (t_fmt): Likewise.
        * localedata/locales/gu_IN (t_fmt): Likewise.
        * localedata/locales/hi_IN (t_fmt): Likewise.
        * localedata/locales/hif_FJ (t_fmt): Likewise.
        * localedata/locales/hne_IN (t_fmt): Likewise.
        * localedata/locales/kn_IN (t_fmt): Likewise.
        * localedata/locales/kok_IN (t_fmt): Likewise.
        * localedata/locales/ks_IN (t_fmt): Likewise.
        * localedata/locales/ks_IN@devanagari (t_fmt): Likewise.
        * localedata/locales/mag_IN (t_fmt): Likewise.
        * localedata/locales/mai_IN (t_fmt): Likewise.
        * localedata/locales/mjw_IN (t_fmt): Likewise.
        * localedata/locales/ml_IN (t_fmt): Likewise.
        * localedata/locales/mni_IN (t_fmt): Likewise.
        * localedata/locales/mr_IN (t_fmt): Likewise.
        * localedata/locales/ms_MY (t_fmt): Likewise.
        * localedata/locales/pa_IN (t_fmt): Likewise.
        * localedata/locales/raj_IN (t_fmt): Likewise.
        * localedata/locales/sa_IN (t_fmt): Likewise.
        * localedata/locales/sat_IN (t_fmt): Likewise.
        * localedata/locales/sd_IN (t_fmt): Likewise.
        * localedata/locales/sd_IN@devanagari (t_fmt): Likewise.
        * localedata/locales/tcy_IN (t_fmt): Likewise.
        * localedata/locales/the_NP (t_fmt): Likewise.
        * localedata/locales/to_TO (t_fmt): Likewise.
        * localedata/locales/ur_IN (t_fmt): Likewise.

        * localedata/locales/hif_FJ (d_t_fmt): Set to
        "%A %d %b %Y %I:%M:%S %p".
        (date_fmt): Add, set to "%A %d %b %Y %I:%M:%S %p %Z".

        * localedata/locales/ar_AE (t_fmt): Set to "%Z %I:%M:%S %p".
        * localedata/locales/ar_BH (t_fmt): Likewise.
        * localedata/locales/ar_DZ (t_fmt): Likewise.
        * localedata/locales/ar_EG (t_fmt): Likewise.
        * localedata/locales/ar_IQ (t_fmt): Likewise.
        * localedata/locales/ar_JO (t_fmt): Likewise.
        * localedata/locales/ar_KW (t_fmt): Likewise.
        * localedata/locales/ar_LB (t_fmt): Likewise.
        * localedata/locales/ar_LY (t_fmt): Likewise.
        * localedata/locales/ar_OM (t_fmt): Likewise.
        * localedata/locales/ar_QA (t_fmt): Likewise.
        * localedata/locales/ar_SD (t_fmt): Likewise.
        * localedata/locales/ar_SS (t_fmt): Likewise.
        * localedata/locales/ar_SY (t_fmt): Likewise.
        * localedata/locales/ar_TN (t_fmt): Likewise.
        * localedata/locales/ar_YE (t_fmt): Likewise.

        * localedata/locales/gez_ER (t_fmt): Set to "%l:%M:%S<U1361>%p".
        (t_fmt_ampm): Likewise.
        * localedata/locales/gez_ET (t_fmt): Likewise.
        (t_fmt_ampm): Likewise.

        * localedata/locales/ta_IN (t_fmt): Set to "%p %I:%M:%S %Z".
        (t_fmt_ampm): Likewise.
        (d_t_fmt): Set to "%A %d %B %Y %p %I:%M:%S %Z".

        * localedata/locales/zh_HK (t_fmt):
        Set to "%p %I<U6642>%M<U5206>%S<U79D2> %Z".

        * localedata/locales/ar_MA (t_fmt_ampm): Set to "" (empty string)
        because this locale does not use the 12-hour clock.
        (t_fmt): Set to "%Z %H:%M:%S".
        (d_t_fmt): Set to "%d %b, %Y %Z %H:%M:%S".

        * localedata/locales/mt_MT (t_fmt_ampm): Set to "" (empty string)
        because this locale does not use the 12-hour clock.
        (t_fmt): Set to "%H:%M:%S %Z".
        (d_t_fmt): Set to "%A, %d ta %b, %Y %H:%M:%S %Z".

        * localedata/locales/so_KE (t_fmt_ampm): Set to "" (empty string)
        because this locale does not use the 12-hour clock.
        (t_fmt): Set to "%T".
        (d_t_fmt): Set to "%A, %B %e, %Y %X %Z".
        (date_fmt): Set to "%A, %B %e, %X %Z %Y".

        * localedata/locales/ta_LK (t_fmt_ampm): Set to "" (empty string)
        because this locale does not use the 12-hour clock.
        (t_fmt): Set to "%H:%M:%S %Z".
        (d_t_fmt): Set to "%A %d %B %Y %H:%M:%S %Z".

        * localedata/locales/bg_BG (t_fmt_ampm): Set to "" (empty string)
        because this locale does not use the 12-hour clock.
        * localedata/locales/cs_CZ (t_fmt_ampm): Likewise.
        * localedata/locales/sk_SK (t_fmt_ampm): Likewise.

https://sourceware.org/git/gitweb.cgi?p=glibc.git;h=27841a7d5abe36c48b41b33d39ca8e5e14b9d36f

commit 27841a7d5abe36c48b41b33d39ca8e5e14b9d36f
Author: Rafal Luzynski <digitalfreak@lingonborough.com>
Date:   Tue Oct 9 01:15:33 2018 +0200

    sq_AL: Use the correct date and time formats (bug 10496, 23724).

    Albanian locale uses the 12-hour clock but some time formats did not
    use any AM/PM indicator making the time ambiguous.  This commit adds
    "%p" wherever it was missing.

    It also sets the correct date format because the old "%Y-%b-%d" produced
    rather weird results like "2018-Sht-28".

    All time formats come from CLDR but as few changes have been introduced
    by this commit as possible.  Some articles from MSDN and other available
    online sources have been also taken into account.

        [BZ #10496]
        [BZ #23724]
        * localedata/locales/sq_AL (t_fmt): Set to "%I:%M:%S.%p %Z".
        (t_fmt_ampm): Likewise.
        (d_t_fmt): Set to "%a %-d %b %Y %I:%M:%S.%p".
        (date_fmt): Add, set to "%a %-d %b %Y %I:%M:%S.%p %Z".
        (d_fmt): Set to "%-d.%-m.%y".

-----------------------------------------------------------------------

Summary of changes:
 ChangeLog                           |  142 +++++++++++++++++++++++++++++++++++
 localedata/locales/aa_DJ            |    4 +-
 localedata/locales/aa_ER            |    4 +-
 localedata/locales/aa_ER@saaho      |    4 +-
 localedata/locales/aa_ET            |    4 +-
 localedata/locales/am_ET            |    4 +-
 localedata/locales/anp_IN           |    2 +-
 localedata/locales/ar_AE            |    2 +-
 localedata/locales/ar_BH            |    2 +-
 localedata/locales/ar_DZ            |    2 +-
 localedata/locales/ar_EG            |    2 +-
 localedata/locales/ar_IN            |    2 +-
 localedata/locales/ar_IQ            |    2 +-
 localedata/locales/ar_JO            |    2 +-
 localedata/locales/ar_KW            |    2 +-
 localedata/locales/ar_LB            |    2 +-
 localedata/locales/ar_LY            |    2 +-
 localedata/locales/ar_MA            |    6 +-
 localedata/locales/ar_OM            |    2 +-
 localedata/locales/ar_QA            |    2 +-
 localedata/locales/ar_SD            |    2 +-
 localedata/locales/ar_SS            |    2 +-
 localedata/locales/ar_SY            |    2 +-
 localedata/locales/ar_TN            |    2 +-
 localedata/locales/ar_YE            |    2 +-
 localedata/locales/bg_BG            |    2 +-
 localedata/locales/bhb_IN           |    2 +-
 localedata/locales/bho_IN           |    2 +-
 localedata/locales/bi_VU            |    2 +-
 localedata/locales/bn_BD            |    2 +-
 localedata/locales/bn_IN            |    2 +-
 localedata/locales/brx_IN           |    2 +-
 localedata/locales/byn_ER           |    4 +-
 localedata/locales/cs_CZ            |    2 +-
 localedata/locales/doi_IN           |    2 +-
 localedata/locales/en_HK            |    4 +-
 localedata/locales/en_IN            |    2 +-
 localedata/locales/en_PH            |    2 +-
 localedata/locales/gez_ER           |    4 +-
 localedata/locales/gez_ET           |    4 +-
 localedata/locales/gu_IN            |    2 +-
 localedata/locales/hi_IN            |    2 +-
 localedata/locales/hif_FJ           |    7 +-
 localedata/locales/hne_IN           |    2 +-
 localedata/locales/kn_IN            |    2 +-
 localedata/locales/kok_IN           |    2 +-
 localedata/locales/ks_IN            |    2 +-
 localedata/locales/ks_IN@devanagari |    2 +-
 localedata/locales/mag_IN           |    2 +-
 localedata/locales/mai_IN           |    2 +-
 localedata/locales/mjw_IN           |    2 +-
 localedata/locales/ml_IN            |    2 +-
 localedata/locales/mni_IN           |    2 +-
 localedata/locales/mr_IN            |    2 +-
 localedata/locales/ms_MY            |    2 +-
 localedata/locales/mt_MT            |    6 +-
 localedata/locales/om_ET            |    4 +-
 localedata/locales/pa_IN            |    2 +-
 localedata/locales/raj_IN           |    2 +-
 localedata/locales/sa_IN            |    2 +-
 localedata/locales/sat_IN           |    2 +-
 localedata/locales/sd_IN            |    2 +-
 localedata/locales/sd_IN@devanagari |    2 +-
 localedata/locales/sid_ET           |    4 +-
 localedata/locales/sk_SK            |    2 +-
 localedata/locales/so_DJ            |    4 +-
 localedata/locales/so_ET            |    4 +-
 localedata/locales/so_KE            |    8 +-
 localedata/locales/so_SO            |    4 +-
 localedata/locales/sq_AL            |   11 ++-
 localedata/locales/ta_IN            |    6 +-
 localedata/locales/ta_LK            |    6 +-
 localedata/locales/tcy_IN           |    2 +-
 localedata/locales/the_NP           |    2 +-
 localedata/locales/ti_ER            |    4 +-
 localedata/locales/ti_ET            |    4 +-
 localedata/locales/tig_ER           |    4 +-
 localedata/locales/to_TO            |    4 +-
 localedata/locales/ur_IN            |    2 +-
 localedata/locales/wal_ET           |    4 +-
 localedata/locales/zh_HK            |    4 +-
 81 files changed, 262 insertions(+), 116 deletions(-)

-- 
You are receiving this mail because:
You are on the CC list for the bug.

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [Bug localedata/23724] Albanian date formats are incorrect
  2018-09-28 20:54 [Bug localedata/23724] New: Albanian date formats are incorrect digitalfreak at lingonborough dot com
                   ` (2 preceding siblings ...)
  2018-12-28 21:18 ` cvs-commit at gcc dot gnu.org
@ 2018-12-28 21:23 ` digitalfreak at lingonborough dot com
  3 siblings, 0 replies; 5+ messages in thread
From: digitalfreak at lingonborough dot com @ 2018-12-28 21:23 UTC (permalink / raw)
  To: libc-locales

https://sourceware.org/bugzilla/show_bug.cgi?id=23724

Rafal Luzynski <digitalfreak at lingonborough dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|---                         |FIXED
   Target Milestone|---                         |2.29

--- Comment #4 from Rafal Luzynski <digitalfreak at lingonborough dot com> ---
Fixed by the commit
https://sourceware.org/git/?p=glibc.git;a=commitdiff;h=27841a7d5abe36c48b41b33d39ca8e5e14b9d36f

-- 
You are receiving this mail because:
You are on the CC list for the bug.

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2018-12-28 21:23 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-09-28 20:54 [Bug localedata/23724] New: Albanian date formats are incorrect digitalfreak at lingonborough dot com
2018-09-28 21:18 ` [Bug localedata/23724] " fweimer at redhat dot com
2018-10-01 10:38 ` digitalfreak at lingonborough dot com
2018-12-28 21:18 ` cvs-commit at gcc dot gnu.org
2018-12-28 21:23 ` digitalfreak at lingonborough dot com

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).