# [PATCH] config: correct log.showRoot description 2 messages from 2022-06-14 to 2022-06-15. Participants: Michael J Gruber, Taylor Blau. Thread: https://gitlist.dev/t/58001 ## Michael J Gruber, 2022-06-14 16:14 Subject: [PATCH] config: correct log.showRoot description Message-ID: URL: https://gitlist.dev/e/b7aa801c3995fffdce37d90adba9253485b616e3.1655223201.git.git%40grubix.eu ``` 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 --- 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 ``` ## Taylor Blau, 2022-06-15 01:50 Subject: Re: [PATCH] config: correct log.showRoot description Message-ID: URL: https://gitlist.dev/e/Yqk66CTVbX22Xi%2Be%40nand.local In-Reply-To: ``` 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 ```