[PATCH] blame: fix typo in documentation

As can be seen in files "Documentation/blame-options.txt" and
"builtin/blame.c", the name of this configuration variable is
"blame.markUnblamableLines".

Signed-off-by: Andrei Rybak <redacted>
---
 Documentation/config/blame.txt | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Documentation/config/blame.txt b/Documentation/config/blame.txt
index 9468e8599c..4d047c1790 100644
--- a/Documentation/config/blame.txt
+++ b/Documentation/config/blame.txt

@@ -27,7 +27,7 @@ blame.ignoreRevsFile::
 	file names will reset the list of ignored revisions.  This option will
 	be handled before the command line option `--ignore-revs-file`.
 
-blame.markUnblamables::
+blame.markUnblamableLines::
 	Mark lines that were changed by an ignored revision that we could not
 	attribute to another commit with a '*' in the output of
 	linkgit:git-blame[1].

-- 
2.32.0

Hi,

On 19/06/21 05.11, Andrei Rybak wrote:

That above is not typo fixing; that is correcting configuration variable 
name.

PS: I have to download mbox of this thread and reply from it because I 
didn't receive the thread on my Thunderbird, regardless of CCing Git ML.

-- 
An old man doll... just what I always wanted! - Clara

On 19/06/2021 07:27, Bagas Sanjaya wrote:

On 19/06/21 05.11, Andrei Rybak wrote:

quoted

As can be seen in files "Documentation/blame-options.txt" and
"builtin/blame.c", the name of this configuration variable is
"blame.markUnblamableLines".

Signed-off-by: Andrei Rybak <redacted>
---
  Documentation/config/blame.txt | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Documentation/config/blame.txt 

b/Documentation/config/blame.txt
index 9468e8599c..4d047c1790 100644

--- a/Documentation/config/blame.txt
+++ b/Documentation/config/blame.txt

@@ -27,7 +27,7 @@ blame.ignoreRevsFile::

      file names will reset the list of ignored revisions.  This 
option will
      be handled before the command line option `--ignore-revs-file`.
-blame.markUnblamables::
+blame.markUnblamableLines::
      Mark lines that were changed by an ignored revision that we 
could not
      attribute to another commit with a '*' in the output of
      linkgit:git-blame[1].

That above is not typo fixing; that is correcting configuration variable 
name.

This issue in documentation of git-blame was introduced when the config 
option
was added in commit 8934ac8c92 (blame: add config options for the output of
ignored or unblamable lines, 2019-05-15).  It first appeared in v6 of that
series [1] where it was called blame.maskIgnoredUnblamables.  In v7 [2] 
it got
renamed to blame.markUnblamableLines.  v7 of that series is the first 
time when
incorrect blame.markUnblamables appears in Documentation/config/blame.txt.

As far as I can tell, it's most probable that Barret Rhoden just missed one
place when renaming this config option between v6 and v7.

If we shouldn't refer to this patch as typofix, how about

	blame: fix name of blame.markUnblamableLines in docs

or

	blame: correct name of config option in docs

or

	config/blame.txt: correct name of blame.markUnblamableLines

?

[1] https://lore.kernel.org/git/20190410162409.117264-5-brho@google.com/
[2] https://lore.kernel.org/git/20190515214503.77162-6-brho@google.com/

On 19/06/21 18.12, Andrei Rybak wrote:

If we shouldn't refer to this patch as typofix, how about

     blame: fix name of blame.markUnblamableLines in docs

or

     blame: correct name of config option in docs

or

     config/blame.txt: correct name of blame.markUnblamableLines

?

I prefer the second, because I prefer not to reveal actual config name 
so that readers must see the whole patch to see the name.

-- 
An old man doll... just what I always wanted! - Clara

As can be seen in files "Documentation/blame-options.txt" and
"builtin/blame.c", the name of this configuration option is
"blame.markUnblamableLines".

Signed-off-by: Andrei Rybak <redacted>
---

Thanks to Bagas Sanjaya for review.  Changes since v1:

  - tweaked commit message

Range-diff:

1:  e32c298ee8 ! 1:  6ea64765e7 blame: fix typo in documentation
    @@ Metadata
     Author: Andrei Rybak [off-list ref]
     
      ## Commit message ##
    -    blame: fix typo in documentation
    +    blame: correct name of config option in docs
     
         As can be seen in files "Documentation/blame-options.txt" and
    -    "builtin/blame.c", the name of this configuration variable is
    +    "builtin/blame.c", the name of this configuration option is
         "blame.markUnblamableLines".
     
         Signed-off-by: Andrei Rybak [off-list ref]

 Documentation/config/blame.txt | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Documentation/config/blame.txt b/Documentation/config/blame.txt
index 9468e8599c..4d047c1790 100644
--- a/Documentation/config/blame.txt
+++ b/Documentation/config/blame.txt

@@ -27,7 +27,7 @@ blame.ignoreRevsFile::
 	file names will reset the list of ignored revisions.  This option will
 	be handled before the command line option `--ignore-revs-file`.
 
-blame.markUnblamables::
+blame.markUnblamableLines::
 	Mark lines that were changed by an ignored revision that we could not
 	attribute to another commit with a '*' in the output of
 	linkgit:git-blame[1].

-- 
2.32.0

Bagas Sanjaya wrote:

That above is not typo fixing; that is correcting configuration variable 
name.

PS: I have to download mbox of this thread and reply from it because I 
didn't receive the thread on my Thunderbird, regardless of CCing Git ML.

This is something I've noticed; I can always see mails on the archive,
however, I don't receive them on my inbox until much later.

-- 
Felipe Contreras

On 19/06/21 18.29, Andrei Rybak wrote:

Looks OK, thanks for contributing.

Reviewed-by: Bagas Sanjaya <redacted>

-- 
An old man doll... just what I always wanted! - Clara

On 6/20/21 5:37 AM, Bagas Sanjaya wrote:

On 19/06/21 18.29, Andrei Rybak wrote:

quoted

As can be seen in files "Documentation/blame-options.txt" and
"builtin/blame.c", the name of this configuration option is
"blame.markUnblamableLines".

Signed-off-by: Andrei Rybak <redacted>

Looks OK, thanks for contributing.

Reviewed-by: Bagas Sanjaya <redacted>

thanks for the fix.  =)

First patch in this series is the same as the v2 patch.  Second patch fixes
similar typos in mentions of config options in test messages.  Third patch is
just a fix for an unrelated typo I've stumbled upon.

Andrei Rybak (3):
  blame: correct name of config option in docs
  t: fix typos in test messages
  git-worktree.txt: fix typo in example path

 Documentation/config/blame.txt | 2 +-
 Documentation/git-worktree.txt | 2 +-
 t/t4258-am-quoted-cr.sh        | 2 +-
 t/t9001-send-email.sh          | 2 +-
 4 files changed, 4 insertions(+), 4 deletions(-)

-- 
2.32.0

As can be seen in files "Documentation/blame-options.txt" and
"builtin/blame.c", the name of this configuration option is
"blame.markUnblamableLines".

Signed-off-by: Andrei Rybak <redacted>
Reviewed-by: Bagas Sanjaya <redacted>
---
 Documentation/config/blame.txt | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Documentation/config/blame.txt b/Documentation/config/blame.txt
index 9468e8599c..4d047c1790 100644
--- a/Documentation/config/blame.txt
+++ b/Documentation/config/blame.txt

@@ -27,7 +27,7 @@ blame.ignoreRevsFile::
 	file names will reset the list of ignored revisions.  This option will
 	be handled before the command line option `--ignore-revs-file`.
 
-blame.markUnblamables::
+blame.markUnblamableLines::
 	Mark lines that were changed by an ignored revision that we could not
 	attribute to another commit with a '*' in the output of
 	linkgit:git-blame[1].

-- 
2.32.0

Both in t4258 and in t9001, the code of the tests following shows the
proper name for the configuration variables.  So use the correct names
in the test messages as well.

Signed-off-by: Andrei Rybak <redacted>
---
 t/t4258-am-quoted-cr.sh | 2 +-
 t/t9001-send-email.sh   | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/t/t4258-am-quoted-cr.sh b/t/t4258-am-quoted-cr.sh
index fb5071f914..201915b45a 100755
--- a/t/t4258-am-quoted-cr.sh
+++ b/t/t4258-am-quoted-cr.sh

@@ -26,7 +26,7 @@ test_expect_success 'am --quoted-cr=strip' '
 	git diff --exit-code HEAD two
 '
 
-test_expect_success 'am with config mailinfo.quotecr=strip' '
+test_expect_success 'am with config mailinfo.quotedCr=strip' '
 	test_might_fail git am --abort &&
 	git reset --hard one &&
 	test_config mailinfo.quotedCr strip &&

diff --git a/t/t9001-send-email.sh b/t/t9001-send-email.sh
index c66d9190de..9ec7d75f0f 100755
--- a/t/t9001-send-email.sh
+++ b/t/t9001-send-email.sh

@@ -1839,7 +1839,7 @@ test_expect_success $PREREQ 'sendemail.aliasfiletype=mailrc' '
 	grep "^!somebody@example\.org!$" commandline1
 '
 
-test_expect_success $PREREQ 'sendemail.aliasfile=~/.mailrc' '
+test_expect_success $PREREQ 'sendemail.aliasesfile=~/.mailrc' '
 	clean_fake_sendmail &&
 	echo "alias sbd  someone@example.org" >"$HOME/.mailrc" &&
 	git config --replace-all sendemail.aliasesfile "~/.mailrc" &&

-- 
2.32.0

Signed-off-by: Andrei Rybak <redacted>
---
 Documentation/git-worktree.txt | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Documentation/git-worktree.txt b/Documentation/git-worktree.txt
index f1bb1fa5f5..66e67e6cbf 100644
--- a/Documentation/git-worktree.txt
+++ b/Documentation/git-worktree.txt

@@ -387,7 +387,7 @@ These annotations are:
 ------------
 $ git worktree list
 /path/to/linked-worktree    abcd1234 [master]
-/path/to/locked-worktreee   acbd5678 (brancha) locked
+/path/to/locked-worktree    acbd5678 (brancha) locked
 /path/to/prunable-worktree  5678abc  (detached HEAD) prunable
 ------------
 

-- 
2.32.0

not sure if this patch really belongs in this specific patchset, but it
might be worth also fixing the same typo in grep as shown by :

--- >8 ---

Subject: grep: fix worktree mis spelling

1d1729caeb (grep: replace grep_read_mutex by internal obj read lock,
 2020-01-15) adds it as part of a comment

fix it to the right spelling without the extra trailing 'e'.

diff --git a/grep.c b/grep.c
index 8f91af1cb0..f40c734565 100644
--- a/grep.c
+++ b/grep.c

@@ -1510,7 +1510,7 @@ static int fill_textconv_grep(struct repository *r,
 	 *
 	 * TODO: allowing text conversion to run in parallel with object
 	 * reading operations might increase performance in the multithreaded
-	 * non-worktreee git-grep with --textconv.
+	 * non-worktree git-grep with --textconv.
 	 */
 	obj_read_lock();
 	size = fill_textconv(r, driver, df, &buf);

Signed-off-by: Carlo Marcelo Arenas Belón <redacted>

On 26/06/2021 08:58, Carlo Marcelo Arenas Belón wrote:

not sure if this patch really belongs in this specific patchset

Yes, third patch is unrelated to the first two.

s/mis spelling/misspelling/

I deliberately skipped this typo, because it is in a TODO comment, which
I thought isn't worth fixing. I probably should have mentioned that in
the patch I've sent.

  	 */
  	obj_read_lock();
  	size = fill_textconv(r, driver, df, &buf);

Signed-off-by: Carlo Marcelo Arenas Belón <redacted>

On 26/06/21 02.38, Andrei Rybak wrote:

First patch in this series is the same as the v2 patch.  Second patch fixes
similar typos in mentions of config options in test messages.  Third patch is
just a fix for an unrelated typo I've stumbled upon.

Andrei Rybak (3):
   blame: correct name of config option in docs
   t: fix typos in test messages
   git-worktree.txt: fix typo in example path

I suggest that the third patch should be on separate single patch, as it 
is unrelated to two other patches that correct config option names.

Thanks.

-- 
An old man doll... just what I always wanted! - Clara

Andrei Rybak [off-list ref] writes:

On 26/06/2021 08:58, Carlo Marcelo Arenas Belón wrote:

quoted

not sure if this patch really belongs in this specific patchset

Yes, third patch is unrelated to the first two.

quoted

, but it
might be worth also fixing the same typo in grep as shown by :

--- >8 ---

Subject: grep: fix worktree mis spelling

s/mis spelling/misspelling/

quoted

1d1729caeb (grep: replace grep_read_mutex by internal obj read lock,
  2020-01-15) adds it as part of a comment
fix it to the right spelling without the extra trailing 'e'.

diff --git a/grep.c b/grep.c
index 8f91af1cb0..f40c734565 100644
--- a/grep.c
+++ b/grep.c

@@ -1510,7 +1510,7 @@ static int fill_textconv_grep(struct repository *r,
  	 *
  	 * TODO: allowing text conversion to run in parallel with object
  	 * reading operations might increase performance in the multithreaded
-	 * non-worktreee git-grep with --textconv.
+	 * non-worktree git-grep with --textconv.

I deliberately skipped this typo, because it is in a TODO comment, which
I thought isn't worth fixing. I probably should have mentioned that in
the patch I've sent.

All sound sensible.  Will queue these three patches as-is.

Thanks.