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

Re: [RFC 0/5] Date Mode: Add --time-zone; deprecate --date=local

From
Michael Witten <mfwitten@gmail.com>
Date
Apr 20, 2011, 14:22 UTC
Message-ID
<0e899fff-5833-4686-aa81-4bd77f2bd633-mfwitten@gmail.com>
In-Reply-To
<20110420064318.GF28597@sigill.intra.peff.net>
On Wed, 20 Apr 2011 02:43:18 -0400, Jeff King wrote:
Show 9 quoted lines
> On Wed, Apr 20, 2011 at 02:53:36AM +0000, Michael Witten wrote:
>
>> The date format `local' has thus been deprecated, though it is still
>> supported (with a warning on stderr).
>
> I don't know if we need to go that far. We can leave it forever as a
> historical compatibility feature. Its existence is not really hurting
> anyone as long as the documentation marks it as deprecated (or doesn't
> even mention it).

The references to a `local' *format* have indeed been removed from the documentation, and the warning is just an indication that the code that supports it will also be removed at some point too.

However, the warning mechanism (in general) could be improved by producing the message only when exiting the program (after the pager has been destroyed); currently, the message can be quite easy to miss otherwise (depending on the way the terminal handles the buffer history).

I don't see the point of leaving crufty code in place forever; it's ridiculous that my modern CPU starts up like a 35 year old chip when the very few people who need that functionality (if anybody) can emulate it if necessary.

As an aside, I've always found it amusing that most programmers are adamant about justifying an existing use case for a new feature before even considering an implementation for it, but yet they will gladly keep around dead code until the next Big Bang.

Previous: Michael WittenNext: Michael Witten
Message 28 of 37 in “[RFC 0/5] Date Mode: Add --time-zone; deprecate --date=local”
  1. Michael WittenApr 20, 2011
  2. 1/5 Light refactoring of date infrastructureMichael Witten, Apr 20, 2011
  3. 2/5 Pretty Print: show tz when using DATE_LOCALMichael Witten, Apr 20, 2011
  4. Junio C HamanoApr 21, 2011
  5. Dates in Commits and other issues of style (Re: [RFC 2/5] Pretty Print: show tz when using DATE_LOCAL)Michael Witten, Apr 22, 2011
  6. Miles BaderApr 25, 2011
  7. Junio C HamanoApr 25, 2011
  8. Jakub NarebskiApr 25, 2011
  9. Junio C HamanoApr 25, 2011
  10. Michael WittenApr 22, 2011
  11. Junio C HamanoApr 22, 2011
  12. 3/5 Date Mode: ImplementationMichael Witten, Apr 20, 2011
  13. Junio C HamanoApr 21, 2011
  14. 4/5 Date Mode: DocumentationMichael Witten, Apr 20, 2011
  15. 5/5 Date Mode: TestsMichael Witten, Apr 20, 2011
  16. Junio C HamanoApr 21, 2011
  17. Michael WittenApr 23, 2011
  18. Michael WittenApr 23, 2011
  19. Time zone option name (Re: [RFC 5/5] Date Mode: Tests)Michael Witten, Apr 23, 2011
  20. Junio C HamanoApr 23, 2011
  21. Michael WittenApr 23, 2011
  22. Jeff KingApr 20, 2011
  23. Michael WittenApr 20, 2011
  24. Junio C HamanoApr 21, 2011
  25. Michael WittenApr 21, 2011
  26. Junio C HamanoApr 21, 2011
  27. Michael WittenApr 21, 2011
  28. Michael WittenApr 20, 2011
  29. Michael WittenApr 20, 2011
  30. Michael WittenApr 20, 2011
  31. Tabs and spaces (Re: [RFC 0/5] Date Mode: Add --time-zone; deprecate --date=local)Jonathan Nieder, Apr 21, 2011
  32. Michael WittenApr 21, 2011
  33. Jonathan NiederApr 21, 2011
  34. Michael WittenApr 21, 2011
  35. Thiago FarinaApr 21, 2011
  36. Alex RiesenApr 21, 2011
  37. Michael WittenApr 21, 2011

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.