* [PATCH] Documentation: fix typos describing date format
@ 2024-04-12 8:51 blanet via GitGitGadget
2024-04-12 16:02 ` Junio C Hamano
0 siblings, 1 reply; 2+ messages in thread
From: blanet via GitGitGadget @ 2024-04-12 8:51 UTC (permalink / raw)
To: git; +Cc: blanet, Xing Xin
From: Xing Xin <xingxin.xx@bytedance.com>
This commit corrects a typographical error found in both
date-formats.txt and git-fast-import.txt documentation, where the term
`email format` was mistakenly used instead of `date format`.
Signed-off-by: Xing Xin <xingxin.xx@bytedance.com>
---
Documentation: fix typos describing date format
Published-As: https://github.com/gitgitgadget/git/releases/tag/pr-1716%2Fblanet%2Fxx%2Fdocumentation-typofix-v1
Fetch-It-Via: git fetch https://github.com/gitgitgadget/git pr-1716/blanet/xx/documentation-typofix-v1
Pull-Request: https://github.com/gitgitgadget/git/pull/1716
Documentation/date-formats.txt | 2 +-
Documentation/git-fast-import.txt | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/Documentation/date-formats.txt b/Documentation/date-formats.txt
index 67645cae64f..e24517c496f 100644
--- a/Documentation/date-formats.txt
+++ b/Documentation/date-formats.txt
@@ -11,7 +11,7 @@ Git internal format::
For example CET (which is 1 hour ahead of UTC) is `+0100`.
RFC 2822::
- The standard email format as described by RFC 2822, for example
+ The standard date format as described by RFC 2822, for example
`Thu, 07 Apr 2005 22:13:13 +0200`.
ISO 8601::
diff --git a/Documentation/git-fast-import.txt b/Documentation/git-fast-import.txt
index b2607366b91..0ccede255ea 100644
--- a/Documentation/git-fast-import.txt
+++ b/Documentation/git-fast-import.txt
@@ -303,7 +303,7 @@ and some sanity checks on the numeric values may also be performed.
with e.g. bogus timezone values.
`rfc2822`::
- This is the standard email format as described by RFC 2822.
+ This is the standard date format as described by RFC 2822.
+
An example value is ``Tue Feb 6 11:22:18 2007 -0500''. The Git
parser is accurate, but a little on the lenient side. It is the
base-commit: 436d4e5b14df49870a897f64fe92c0ddc7017e4c
--
gitgitgadget
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] Documentation: fix typos describing date format
2024-04-12 8:51 [PATCH] Documentation: fix typos describing date format blanet via GitGitGadget
@ 2024-04-12 16:02 ` Junio C Hamano
0 siblings, 0 replies; 2+ messages in thread
From: Junio C Hamano @ 2024-04-12 16:02 UTC (permalink / raw)
To: blanet via GitGitGadget; +Cc: git, blanet, Xing Xin
"blanet via GitGitGadget" <gitgitgadget@gmail.com> writes:
> From: Xing Xin <xingxin.xx@bytedance.com>
>
> This commit corrects a typographical error found in both
> date-formats.txt and git-fast-import.txt documentation, where the term
> `email format` was mistakenly used instead of `date format`.
Saying "date" is much more correct than "email" ;-). Thanks for
noticing both instances.
> Documentation/date-formats.txt | 2 +-
> Documentation/git-fast-import.txt | 2 +-
> 2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/Documentation/date-formats.txt b/Documentation/date-formats.txt
> index 67645cae64f..e24517c496f 100644
> --- a/Documentation/date-formats.txt
> +++ b/Documentation/date-formats.txt
> @@ -11,7 +11,7 @@ Git internal format::
> For example CET (which is 1 hour ahead of UTC) is `+0100`.
>
> RFC 2822::
> - The standard email format as described by RFC 2822, for example
> + The standard date format as described by RFC 2822, for example
> `Thu, 07 Apr 2005 22:13:13 +0200`.
RFC 2822 defines it as "date-time" (and this hasn't been updated by
RFC 5322). "date" is used to specifically mean "day month year".
https://www.rfc-editor.org/rfc/rfc2822.html#section-3.3
date-time = [ day-of-week "," ] date FWS time [CFWS]
day-of-week = ([FWS] day-name) / obs-day-of-week
day-name = "Mon" / "Tue" / "Wed" / "Thu" /
"Fri" / "Sat" / "Sun"
date = day month year
So the new phrasing may get complain from pedants. Saying "The
standard date-time format as described by RFC 2822" is more
technically correct, but I actually wonder if we should go in the
opposite, looser direction, like
The timestamp format used by RFC 2822, for example
...
so that we do not even pretend to use the wording they use in the
RFC documents. Doing so makes it in line with "git rev-list --help"
which says
"--date=rfc" shows timestamps in RFC 2822 format, often
found in e-mail messages.
But that's minor. "date" is fine as-is. The same comment applies
to the other hunk.
Thanks.
> diff --git a/Documentation/git-fast-import.txt b/Documentation/git-fast-import.txt
> index b2607366b91..0ccede255ea 100644
> --- a/Documentation/git-fast-import.txt
> +++ b/Documentation/git-fast-import.txt
> @@ -303,7 +303,7 @@ and some sanity checks on the numeric values may also be performed.
> with e.g. bogus timezone values.
>
> `rfc2822`::
> - This is the standard email format as described by RFC 2822.
> + This is the standard date format as described by RFC 2822.
> +
> An example value is ``Tue Feb 6 11:22:18 2007 -0500''. The Git
> parser is accurate, but a little on the lenient side. It is the
>
> base-commit: 436d4e5b14df49870a897f64fe92c0ddc7017e4c
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2024-04-12 16:02 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-04-12 8:51 [PATCH] Documentation: fix typos describing date format blanet via GitGitGadget
2024-04-12 16:02 ` Junio C Hamano
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).