public inbox for glibc-cvs@sourceware.org
help / color / mirror / Atom feed
* [glibc] en_US: Minimize changes to date_fmt (Bug 25923)
@ 2020-07-17 13:17 Carlos O'Donell
  0 siblings, 0 replies; only message in thread
From: Carlos O'Donell @ 2020-07-17 13:17 UTC (permalink / raw)
  To: glibc-cvs

https://sourceware.org/git/gitweb.cgi?p=glibc.git;h=8cde977077b3568310c743b21a905ca9ab286724

commit 8cde977077b3568310c743b21a905ca9ab286724
Author: Carlos O'Donell <carlos@redhat.com>
Date:   Wed Jul 8 15:58:06 2020 -0400

    en_US: Minimize changes to date_fmt (Bug 25923)
    
    In 2000 when date_fmt was originally added as an extension the
    en_US locale did not have a date_fmt specifier and so used the
    default which resulted in the abbreviated month name coming
    before the day of the month (as expected in the US and other
    locales).  In commit 7395f3a0efad9fc51bb54fa383ef6524702e0c49 the
    date_fmt was added to en_US with a 12H time to better align with
    US user expectations.  Unfortunately the abbreviated month name
    and day were inverted during that transition, and that was seen
    as a regression and reported against Fedora 32:
    https://bugzilla.redhat.com/show_bug.cgi?id=1830623
    
    The progression of date_fmt looks like this:
    "%a %b %e %H:%M:%S %Z %Y"    <- Originally (2000)
    "%a %d %b %Y %I:%M:%S %p %Z" <- glibc 2.29 (2019)
    "%a %b %e %r %Z %Y"          <- glibc 2.32 (2020) [this commit]
    
    Note: "%r" is "%I:%M:%S %p" in en_US and so shorter to write.
    
    Likewise the year is in the wrong place in commit
    7395f3a0efad9fc51bb54fa383ef6524702e0c49 and this is corrected in
    this patch.
    
    For reference d_t_fmt:
    "%a %d %b %Y %r %Z"          <- d_t_fmt    (1997)
    
    Yes, d_t_fmt and date_fmt are *not* the same, this is just the
    history of this locale. This commit does not change d_t_fmt to
    better align with date_fmt. No users have requested we change
    d_t_fmt or given any justification for such a change.
    
    The only goals of this change are to place the abbreviated month
    name before the day of the month as it has been printed since
    2000, and place the year at the end. This minimizes the change
    from commit 7395f3a0efad9fc51bb54fa383ef6524702e0c49 and makes
    good on changing only from 24H clock to 12H clock.
    
    Reviewed-by: Florian Weimer <fweimer@redhat.com>

Diff:
---
 localedata/locales/en_US | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/localedata/locales/en_US b/localedata/locales/en_US
index e71abf5d5c..15c6354fcf 100644
--- a/localedata/locales/en_US
+++ b/localedata/locales/en_US
@@ -117,8 +117,11 @@ t_fmt   "%r"
 % Appropriate AM/PM time representation (%r)
 t_fmt_ampm "%I:%M:%S %p"
 %
-% Appropriate date and time representation for date(1)
-date_fmt "%a %d %b %Y %r %Z"
+% Appropriate date and time representation for date(1).  This is
+% different from d_t_fmt for historical reasons and has been different
+% since 2000 when date_fmt was added as a GNU extension.  At the end
+% of 2018 it was adjusted to use 12H time (bug 24046) instead of 24H.
+date_fmt "%a %b %e %r %Z %Y"
 %
 % Strings for AM/PM
 %


^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2020-07-17 13:17 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-07-17 13:17 [glibc] en_US: Minimize changes to date_fmt (Bug 25923) Carlos O'Donell

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).