git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [BUG REPORT] Git log pretty date

From
Jeff King <peff@peff.net>
Date
Jun 3, 2014, 03:46 UTC
Message-ID
<20140603034619.GA18176@sigill.intra.peff.net>
In-Reply-To
<CACJZPBtJZ+gDvtHnRwi8JOO7rFAg=pPBvEQ=-FwKnp9Gr+FStw@mail.gmail.com>
On Fri, May 30, 2014 at 09:08:57AM +0100, Rodrigo Fernandes wrote:
> Do you have any idea how does github understand that is a bug and
> fixes it automatically?
> (I'm saying this because on Github the date is correct).

I looked into this. The dates you see on GitHub's web UI are actually parsed by Rugged/libgit2. The libgit2 parser is slightly more forgiving in this instance; if it sees a broken timezone, it will leave the timestamp intact, and only omit the timezone. Whereas git says "no, it's broken, and the timestamp cannot be trusted".

I think both are equally valid strategies, and I do not even think it is a problem that they diverge between the two implementations. I'd be OK with a patch to make git handle errors in each independently, assuming it is not too invasive.

-Peff
Previous: Rodrigo FernandesNext: Dennis Kaarsemaker
Message 8 of 13 in “[BUG REPORT] Git log pretty date”
  1. Rodrigo FernandesMay 29, 2014
  2. Duy NguyenMay 29, 2014
  3. Jeff KingMay 29, 2014
  4. Junio C HamanoMay 29, 2014
  5. Jeff KingMay 29, 2014
  6. Rodrigo FernandesMay 29, 2014
  7. Rodrigo FernandesMay 30, 2014
  8. Jeff KingJun 3, 2014
  9. Dennis KaarsemakerJun 3, 2014
  10. Jeff KingJun 3, 2014
  11. Dennis KaarsemakerMay 29, 2014
  12. Rodrigo FernandesMay 29, 2014
  13. Duy NguyenMay 29, 2014

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.