From: john at keeping.me.uk (John Keeping)
Subject: [PATCH 0/8] Show times in the originator's timezone
Date: Mon, 8 Feb 2016 14:08:44 +0000 [thread overview]
Message-ID: <20160208140844.GL29880@serenity.lan> (raw)
In-Reply-To: <CAHmME9piS_8aX-fmq=LArPikW7qRzdJLuhZ-9ybSdU+R08zh_g@mail.gmail.com>
On Mon, Feb 08, 2016 at 02:38:57PM +0100, Jason A. Donenfeld wrote:
> This series is faulty. See git.zx2c4.com
>
> It shows (GMT) when those times are not GMT.
You're right, it seems that the problem is that we end up using
strftime(3) with %Z for the timezone, but that function doesn't take an
explicit timezone so it just ends up using the system one.
I think we should just switch to using DATE_SHORT and DATE_ISO8601 from
Git's date_mode_type enum; I didn't do so originally because the format
doesn't quite match what we have used traditionally, but now that I look
again it's not that different so I reckon we should just switch (it was
a bigger problem before Git learned that "local" is orthogonal to the
display mode).
We may still need a custom strftime(3) string for FMT_ATOMDATE, but
DATE_ISO8601_STRICT may be enough; it depends whether atom dates have to
be in +00:00 or not.
next prev parent reply other threads:[~2016-02-08 14:08 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-01-19 19:33 john
2016-01-19 19:33 ` [PATCH 1/8] ui-shared: remove "format" from cgit_print_age() john
2016-01-19 19:33 ` [PATCH 2/8] parsing: add timezone to ident structures john
2016-01-19 19:33 ` [PATCH 3/8] ui-shared: add cgit_date_mode() john
2016-02-08 13:20 ` Jason
2016-02-08 13:23 ` Jason
2016-01-19 19:33 ` [PATCH 4/8] ui-{commit,tag}: show dates in originator's timezone john
2016-01-19 19:33 ` [PATCH 5/8] ui: show ages in the " john
2016-01-19 19:33 ` [PATCH 6/8] ui-shared: use show_date for footer timestamp john
2016-01-19 19:33 ` [PATCH 7/8] ui-atom: use show_date directly for atom dates john
2016-01-19 19:33 ` [PATCH 8/8] ui-shared: remove cgit_print_date() john
2016-02-08 13:23 ` [PATCH 0/8] Show times in the originator's timezone Jason
2016-02-08 13:38 ` Jason
2016-02-08 14:08 ` john [this message]
2016-02-08 14:33 ` Jason
2016-02-08 15:05 ` [PATCH 1/2] Avoid DATE_STRFTIME for long/short dates john
2016-02-08 15:06 ` [PATCH 2/2] ui-atom: avoid DATE_STRFTIME john
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20160208140844.GL29880@serenity.lan \
--to=cgit@lists.zx2c4.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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 NNTP newsgroup(s).