[PATCH] config: correct log.showRoot description

[PATCH] config: correct log.showRoot description

[Michael J Gruber]

The current description is misleading because it fits a "default false"
setting whereas the default is "true" (as mentioned).

Instead, craft the wording in `config/log.txt` analogous to the one in
`git-log.txt`.

Note that the wording is not exactly the same, e.g. because
`config/log.txt` wants to link to man pages and man pages want to refer
to command options. Otherwise, includes would be the easy route to
consistency here.

Signed-off-by: Michael J Gruber <git@grubix.eu>
---
 Documentation/config/log.txt | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/Documentation/config/log.txt b/Documentation/config/log.txt
index 456eb07800..b210f24cca 100644
--- a/Documentation/config/log.txt
+++ b/Documentation/config/log.txt
@@ -40,10 +40,10 @@ log.graphColors::
 	history lines in `git log --graph`.
 
 log.showRoot::
-	If true, the initial commit will be shown as a big creation event.
-	This is equivalent to a diff against an empty tree.
-	Tools like linkgit:git-log[1] or linkgit:git-whatchanged[1], which
-	normally hide the root commit will now show it. True by default.
+	If `false`, tools like linkgit:git-log[1] or linkgit:git-whatchanged[1]
+	will not treat the initial commit as a big creation event.  Any root
+	commits in `git log -p` output would be shown without a diff attached.
+	The default is `true`.
 
 log.showSignature::
 	If true, makes linkgit:git-log[1], linkgit:git-show[1], and
-- 
2.37.0.rc0.107.g7a7be657e7

Re: [PATCH] config: correct log.showRoot description

[Taylor Blau]

On Tue, Jun 14, 2022 at 06:14:22PM +0200, Michael J Gruber wrote:
> diff --git a/Documentation/config/log.txt b/Documentation/config/log.txt
> index 456eb07800..b210f24cca 100644
> --- a/Documentation/config/log.txt
> +++ b/Documentation/config/log.txt
> @@ -40,10 +40,10 @@ log.graphColors::
>  	history lines in `git log --graph`.
>
>  log.showRoot::
> -	If true, the initial commit will be shown as a big creation event.
> -	This is equivalent to a diff against an empty tree.
> -	Tools like linkgit:git-log[1] or linkgit:git-whatchanged[1], which
> -	normally hide the root commit will now show it. True by default.
> +	If `false`, tools like linkgit:git-log[1] or linkgit:git-whatchanged[1]
> +	will not treat the initial commit as a big creation event.  Any root
> +	commits in `git log -p` output would be shown without a diff attached.
> +	The default is `true`.

This looks good to me. An alternative approach which would work to would
be to replace "If `true`" with "When `true`". I have a vague preference
towards the latter, since it explains what log.showRoot does, rather
than what it does not do.

But this works just as well, and I think that I'm splitting hairs
anyways ;).

Thanks,
Taylor