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

Re: Time zone option name (Re: [RFC 5/5] Date Mode: Tests)

From
Junio C Hamano <gitster@pobox.com>
Date
Apr 23, 2011, 05:27 UTC
Message-ID
<7vfwp9mt9k.fsf@alter.siamese.dyndns.org>
In-Reply-To
<453b72ab-40cc-431b-9865-3080c250a7dd-mfwitten@gmail.com>
Michael Witten <mfwitten@gmail.com> writes:
> I'd like to clarify what you intend the new time zone option to be named; my
> patch series currently uses `--time-zone' (which is admittedly long), but it
> would appear here that you find `--zone' attractive.

Sorry, that was just me being sloppy, writing barely enough for you and readers to be able to tell which one of the two options you used in your patch is what I was referring to.

The word "zone" may mean something other than timezone in future versions of git, and I do not want to see us regret this feature squatting on a short and sweet "--zone" option if/when it happens. Your "--time-zone" (or a single-word "--timezone") would be far better.

The "--date={local,default,short,...}" option might want to have a longer synonym "--date-format={...ditto...}", as some commands take the value of the timestamp as a parameter to their "--date=" option, but that is minor and definitely should not be part of this series.

Thanks.
Previous: Michael WittenNext: Michael Witten
Message 20 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.