git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Marcus Comstedt <marcus@mc.pp.se>
To: git@vger.kernel.org
Subject: PATCH: Improved support for ISO 8601 timezones
Date: Mon, 17 May 2010 21:07:09 +0200	[thread overview]
Message-ID: <1274123231-18482-1-git-send-email-marcus@mc.pp.se> (raw)

Hi.

I discovered that git's date parser does not understand "Z" to mean
the "UTC" timezone.  This is unfortunate, because the use of "Z" is
prescribed by ISO 8601.

I made a small patch to add "Z" as an alias for "UTC", which enables
standard ISO 8601 timestamps to be parsed correctly.  Also, it fixes
a bug that at least three characters of the timezone name had to match,
which is of course impossible when the name of the timezone is shorter
than three characters.  There was already such a timezone before ("NT")
which could not be selected due to the bug.

The second patch, which is perhaps less essential, adds support for
the remaining numerical timezone indicators defined by ISO 8601 not
already supported by git (only +-hhmm was supported, but ISO 8601
also specifies that +-hh:mm and +-hh are ok as well).

Thanks


  // Marcus

             reply	other threads:[~2010-05-17 19:15 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-05-17 19:07 Marcus Comstedt [this message]
2010-05-17 19:07 ` [PATCH 1/2] Added "Z" as an alias for the timezone "UTC" Marcus Comstedt
2010-05-17 20:32   ` Jay Soffian
2010-05-17 19:07 ` [PATCH 2/2] Accept the timezone specifiers [+-]hh:mm and [+-]hh in addition to [+-]hhmm Marcus Comstedt
2010-05-19 14:31   ` Junio C Hamano
2010-05-19 17:21     ` Marcus Comstedt

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=1274123231-18482-1-git-send-email-marcus@mc.pp.se \
    --to=marcus@mc.pp.se \
    --cc=git@vger.kernel.org \
    /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).