* [PATCH] doc: git-update-index: add --show-index-version to synopsis
@ 2024-05-10 18:36 Dov Murik
2024-05-11 3:18 ` Linus Arver
0 siblings, 1 reply; 3+ messages in thread
From: Dov Murik @ 2024-05-10 18:36 UTC (permalink / raw
To: git; +Cc: Dov Murik, Junio C Hamano, Linus Arver
Fixes: 606e088d5d9a ("update-index: add --show-index-version")
Signed-off-by: Dov Murik <dov.murik@linux.dev>
---
Documentation/git-update-index.txt | 1 +
1 file changed, 1 insertion(+)
diff --git a/Documentation/git-update-index.txt b/Documentation/git-update-index.txt
index 8c47890a6a..7128aed540 100644
--- a/Documentation/git-update-index.txt
+++ b/Documentation/git-update-index.txt
@@ -25,6 +25,7 @@ SYNOPSIS
[--really-refresh] [--unresolve] [--again | -g]
[--info-only] [--index-info]
[-z] [--stdin] [--index-version <n>]
+ [--show-index-version]
[--verbose]
[--] [<file>...]
base-commit: 3402c0e53fb798cb471dd6562eb5c938885b7295
--
2.39.3 (Apple Git-146)
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] doc: git-update-index: add --show-index-version to synopsis
2024-05-10 18:36 [PATCH] doc: git-update-index: add --show-index-version to synopsis Dov Murik
@ 2024-05-11 3:18 ` Linus Arver
2024-05-12 1:52 ` Dov Murik
0 siblings, 1 reply; 3+ messages in thread
From: Linus Arver @ 2024-05-11 3:18 UTC (permalink / raw
To: Dov Murik, git; +Cc: Junio C Hamano, Linus Arver
Hi Dov,
Dov Murik <dov.murik@linux.dev> writes:
> Fixes: 606e088d5d9a ("update-index: add --show-index-version")
I don't think this trailer is a standard one. Also, the "fixes" doesn't
really explain much. Perhaps you meant something like
In 606e088d5d (update-index: add --show-index-version, 2023-09-12),
we added the new '--show-index-version' option and documented it,
but forgot to add it to the synopsis section. Do so now.
?
> Signed-off-by: Dov Murik <dov.murik@linux.dev>
> ---
> Documentation/git-update-index.txt | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/Documentation/git-update-index.txt b/Documentation/git-update-index.txt
> index 8c47890a6a..7128aed540 100644
> --- a/Documentation/git-update-index.txt
> +++ b/Documentation/git-update-index.txt
> @@ -25,6 +25,7 @@ SYNOPSIS
> [--really-refresh] [--unresolve] [--again | -g]
> [--info-only] [--index-info]
> [-z] [--stdin] [--index-version <n>]
> + [--show-index-version]
> [--verbose]
> [--] [<file>...]
>
>
> base-commit: 3402c0e53fb798cb471dd6562eb5c938885b7295
> --
> 2.39.3 (Apple Git-146)
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] doc: git-update-index: add --show-index-version to synopsis
2024-05-11 3:18 ` Linus Arver
@ 2024-05-12 1:52 ` Dov Murik
0 siblings, 0 replies; 3+ messages in thread
From: Dov Murik @ 2024-05-12 1:52 UTC (permalink / raw
To: Linus Arver, git; +Cc: Junio C Hamano, Linus Arver, dov.murik
Hi Linus,
Thank you for looking at this patch.
On Fri, May 10, 2024 at 10:18:36PM -0500, Linus Arver wrote:
> Hi Dov,
>
> Dov Murik <dov.murik@linux.dev> writes:
>
> > Fixes: 606e088d5d9a ("update-index: add --show-index-version")
>
> I don't think this trailer is a standard one.
I saw it in Linux kernel patches. From [1]:
A Fixes: tag indicates that the patch fixes an issue in a previous
commit. It is used to make it easy to determine where a bug
originated, which can help review a bug fix. This tag also assists
the stable kernel team in determining which stable kernel versions
should receive your fix. This is the preferred method for indicating
a bug fixed by the patch.
But I now understand it is not used in git patches; sorry about that.
[1] https://www.kernel.org/doc/html/latest/process/submitting-patches.html
> Also, the "fixes" doesn't
> really explain much. Perhaps you meant something like
>
> In 606e088d5d (update-index: add --show-index-version, 2023-09-12),
> we added the new '--show-index-version' option and documented it,
> but forgot to add it to the synopsis section. Do so now.
>
> ?
Yes, the explicit explanation is clearer. Thank you.
I'll submit a v2 with a clearer commit message.
-Dov
>
> > Signed-off-by: Dov Murik <dov.murik@linux.dev>
> > ---
> > Documentation/git-update-index.txt | 1 +
> > 1 file changed, 1 insertion(+)
> >
> > diff --git a/Documentation/git-update-index.txt b/Documentation/git-update-index.txt
> > index 8c47890a6a..7128aed540 100644
> > --- a/Documentation/git-update-index.txt
> > +++ b/Documentation/git-update-index.txt
> > @@ -25,6 +25,7 @@ SYNOPSIS
> > [--really-refresh] [--unresolve] [--again | -g]
> > [--info-only] [--index-info]
> > [-z] [--stdin] [--index-version <n>]
> > + [--show-index-version]
> > [--verbose]
> > [--] [<file>...]
> >
> >
> > base-commit: 3402c0e53fb798cb471dd6562eb5c938885b7295
> > --
> > 2.39.3 (Apple Git-146)
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2024-05-12 1:52 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-05-10 18:36 [PATCH] doc: git-update-index: add --show-index-version to synopsis Dov Murik
2024-05-11 3:18 ` Linus Arver
2024-05-12 1:52 ` Dov Murik
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 read-only IMAP folder(s) and NNTP newsgroup(s).