From: Junio C Hamano <gitster@pobox.com>
To: "Adam Johnson via GitGitGadget" <gitgitgadget@gmail.com>
Cc: git@vger.kernel.org, "Ævar Arnfjörð Bjarmason" <avarab@gmail.com>,
"Matheus Tavares" <matheus.bernardino@usp.br>,
"Adam Johnson" <me@adamj.eu>
Subject: Re: [PATCH] difftool docs: restore correct position of tool list
Date: Thu, 09 Jan 2025 08:46:33 -0800 [thread overview]
Message-ID: <xmqq1pxcj5p2.fsf@gitster.g> (raw)
In-Reply-To: <pull.1849.git.1736379323427.gitgitgadget@gmail.com> (Adam Johnson via GitGitGadget's message of "Wed, 08 Jan 2025 23:35:23 +0000")
"Adam Johnson via GitGitGadget" <gitgitgadget@gmail.com> writes:
> From: Adam Johnson <me@adamj.eu>
>
> 2a9dfdf260 (difftool docs: de-duplicate configuration sections, 2022-09-07)
> moved the difftool documentation, but missed moving this "include" line that
> includes the generated list of diff tools, as referenced in the moved text.
Thanks for a very clearly written problem description.
> diff --git a/Documentation/config/diff.txt b/Documentation/config/diff.txt
> index fdae13a2122..1135a62a0ad 100644
> --- a/Documentation/config/diff.txt
> +++ b/Documentation/config/diff.txt
> @@ -218,8 +218,6 @@ endif::git-diff[]
> Set this option to `true` to make the diff driver cache the text
> conversion outputs. See linkgit:gitattributes[5] for details.
>
> -include::{build_dir}/mergetools-diff.txt[]
> -
Above this include, there was an entry for diff.guitool, and this
generated file was to enumerate possible values for the
configuration.
> `diff.indentHeuristic`::
> Set this option to `false` to disable the default heuristics
> that shift diff hunk boundaries to make patches easier to read.
> diff --git a/Documentation/config/difftool.txt b/Documentation/config/difftool.txt
> index 447c40d85a2..6cd47331a91 100644
> --- a/Documentation/config/difftool.txt
> +++ b/Documentation/config/difftool.txt
> @@ -13,6 +13,8 @@ diff.guitool::
> and requires that a corresponding difftool.<guitool>.cmd variable
> is defined.
>
> +include::{build_dir}/mergetools-diff.txt[]
> +
And the list now sits at the right place.
Will queue. Thanks.
> difftool.<tool>.cmd::
> Specify the command to invoke the specified diff tool.
> The specified command is evaluated in shell with the following
>
> base-commit: a60673e9252b08d4eca90543b3729f4798b9aafd
prev parent reply other threads:[~2025-01-09 16:46 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-01-08 23:35 [PATCH] difftool docs: restore correct position of tool list Adam Johnson via GitGitGadget
2025-01-09 16:46 ` Junio C Hamano [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=xmqq1pxcj5p2.fsf@gitster.g \
--to=gitster@pobox.com \
--cc=avarab@gmail.com \
--cc=git@vger.kernel.org \
--cc=gitgitgadget@gmail.com \
--cc=matheus.bernardino@usp.br \
--cc=me@adamj.eu \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.