threads / patch / 63141

patchdoc: add missing commit C to the graph for --ancestry-path=H D..M

Subject: [PATCH] doc: add missing commit C to the graph for --ancestry-path=H D..M

## tl;dr

3 messages between Mar 16, 2025 and Mar 16, 2025. Diffs are folded; open one to read it.

replies: 2people: 2as markdown or json

Han Jiang via GitGitGadget· Mar 16, 2025, 01:47 UTC · lore
From: Han Jiang <jhcarl0814@gmail.com>
The graph for `--ancestry-path=H D..M` should contain commit C.
Signed-off-by: Han Jiang <jhcarl0814@gmail.com>
---
    doc: add missing commit C to the graph for --ancestry-path=H D..M
    
    cc:Elijah Newren newren@gmail.com,D. Ben Knoble ben.knoble@gmail.com
Published-As: https://github.com/gitgitgadget/git/releases/tag/pr-1883%2Fjhcarl0814%2Fpatch_ancestry_path_doc_graph_20250315-v1
Fetch-It-Via: git fetch https://github.com/gitgitgadget/git pr-1883/jhcarl0814/patch_ancestry_path_doc_graph_20250315-v1
Pull-Request: https://github.com/gitgitgadget/git/pull/1883
 Documentation/rev-list-options.adoc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
Show changes to Documentation/rev-list-options.adoc +1 −1
diff --git a/Documentation/rev-list-options.adoc b/Documentation/rev-list-options.adoc
index 785c0786e0c..5e5cedb9127 100644
--- a/Documentation/rev-list-options.adoc
+++ b/Documentation/rev-list-options.adoc
@@ -640,7 +640,7 @@ commits affected by that topic, we may only want to view the subset of
 -----------------------------------------------------------------------
 		E
 		 \
-		  G---H---I---J
+	      C---G---H---I---J
 			       \
 				L--M
 -----------------------------------------------------------------------

base-commit: 683c54c999c301c2cd6f715c411407c413b1d84e
-- 
gitgitgadget
Elijah Newren· Mar 16, 2025, 06:53 UTC · re: Han Jiang via GitGitGadget · lore

Re: [PATCH] doc: add missing commit C to the graph for --ancestry-path=H D..M

On Sat, Mar 15, 2025 at 6:47 PM Han Jiang via GitGitGadget <gitgitgadget@gmail.com> wrote:

>
> From: Han Jiang <jhcarl0814@gmail.com>
>
> The graph for `--ancestry-path=H D..M` should contain commit C.
Yep, C is in the range D..M and C is also an ancestor of H.
Show 5 quoted lines
> Signed-off-by: Han Jiang <jhcarl0814@gmail.com>
> ---
>     doc: add missing commit C to the graph for --ancestry-path=H D..M
>
>     cc:Elijah Newren newren@gmail.com,D. Ben Knoble ben.knoble@gmail.com

In the future, you'll probably want to use separate cc lines for each person you are cc'ing; I think gitgitgadget expects that to work. You also might want to add a space after the "cc:", though I don't know if that's required.

Show 24 quoted lines
> Published-As: https://github.com/gitgitgadget/git/releases/tag/pr-1883%2Fjhcarl0814%2Fpatch_ancestry_path_doc_graph_20250315-v1
> Fetch-It-Via: git fetch https://github.com/gitgitgadget/git pr-1883/jhcarl0814/patch_ancestry_path_doc_graph_20250315-v1
> Pull-Request: https://github.com/gitgitgadget/git/pull/1883
>
>  Documentation/rev-list-options.adoc | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/Documentation/rev-list-options.adoc b/Documentation/rev-list-options.adoc
> index 785c0786e0c..5e5cedb9127 100644
> --- a/Documentation/rev-list-options.adoc
> +++ b/Documentation/rev-list-options.adoc
> @@ -640,7 +640,7 @@ commits affected by that topic, we may only want to view the subset of
>  -----------------------------------------------------------------------
>                 E
>                  \
> -                 G---H---I---J
> +             C---G---H---I---J
>                                \
>                                 L--M
>  -----------------------------------------------------------------------
>
> base-commit: 683c54c999c301c2cd6f715c411407c413b1d84e
> --
> gitgitgadget
Patch looks good.
Han Jiang· Mar 16, 2025, 09:36 UTC · re: Elijah Newren · lore

Re: [PATCH] doc: add missing commit C to the graph for --ancestry-path=H D..M

On Sun, Mar 16, 2025 at 7:53 PM Elijah Newren <newren@gmail.com> wrote:
Show 5 quoted lines
>
> In the future, you'll probably want to use separate cc lines for each
> person you are cc'ing; I think gitgitgadget expects that to work.  You
> also might want to add a space after the "cc:", though I don't know if
> that's required.
Thank you for your suggestion! I will follow it next time.

← back to recent threads