{"thread":{"id":"56586","subject":"git-blame: '--color-by-age' option not documented in manual page","startedAt":"2021-09-24T19:21:00Z","lastAt":"2021-09-29T18:31:00Z","messageCount":14,"participants":["Dr. Matthias St. Pierre","Bagas Sanjaya","Junio C Hamano"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"437018","messageId":"d918fe0de6b04d8d848050d4aedbe060@ncp-e.com","threadId":"56586","inReplyTo":null,"subject":"git-blame: '--color-by-age' option not documented in manual page","fromName":"Dr. Matthias St. Pierre","fromEmail":"matthias.st.pierre@ncp-e.com","sentAt":"2021-09-24T19:15:40Z","receivedAt":"2021-09-24T19:21:00Z","isPatch":false,"sender":{"key":"matthias.st.pierre@ncp-e.com","avatar":"https://gravatar.com/avatar/b3123d9574916986285f6ac929edb3b776525afd0b4196e61e5126867de764c6?d=mp&s=160"},"body":"Hi,\n\nI just noticed that the '--color-by-age' option of git-blame (introduced in 25d5f52901f0d56b2e3df06b53062ee599b0403b)\nis listed by 'git blame -h' but not documented in the manual page.\n\nRegards,\nMatthias\n\n\n\n\nDr. Matthias St. Pierre\nTech Lead Cryptography\nmatthias.st.pierre@ncp-e.com\nPhone: +49 911 9968-0\n www.ncp-e.com\n\nHeadquarters Germany: NCP engineering GmbH • Dombuehler Str. 2 • 90449 • Nuremberg\nNorth American HQ: NCP engineering Inc. • 601 Cleveland Str., Suite 501-25 • Clearwater, FL 33755\n\nAuthorized representatives: Peter Soell, Patrick Oliver Graf, Beate Dietrich\nRegistry Court: Lower District Court of Nuremberg\nCommercial register No.: HRB 7786 Nuremberg, VAT identification No.: DE 133557619\n\nThis e-mail message including any attachments is for the sole use of the intended recipient(s) and may contain privileged\nor confidential information. Any unauthorized review, use, disclosure or distribution is prohibited. If you are not the intended recipient,\nplease immediately contact the sender by reply e-mail and delete the original message and destroy all copies thereof.\n"},{"id":"437070","messageId":"20210925121817.1089897-1-bagasdotme@gmail.com","threadId":"56586","inReplyTo":"d918fe0de6b04d8d848050d4aedbe060@ncp-e.com","subject":"[PATCH] blame: document --color-* options","fromName":"Bagas Sanjaya","fromEmail":"bagasdotme@gmail.com","sentAt":"2021-09-25T12:18:18Z","receivedAt":"2021-09-25T12:20:11Z","isPatch":true,"sender":{"key":"bagasdotme@gmail.com","avatar":"https://avatars.githubusercontent.com/u/40219486?v=4"},"body":"Commit cdc2d5f11f1a (builtin/blame: dim uninteresting metadata lines,\n2018-04-23) and 25d5f52901f0 (builtin/blame: highlight recently changed\nlines, 2018-04-23) introduce --color-lines and --color-by-age options to\ngit blame, respectively. While both options are mentioned in usage help,\nthey aren't documented in git-blame(1). Document them.\n\nSigned-off-by: Bagas Sanjaya \n---\n Documentation/blame-options.txt | 8 ++++++++\n Documentation/git-blame.txt | 4 ++--\n 2 files changed, 10 insertions(+), 2 deletions(-)\n\ndiff --git a/Documentation/blame-options.txt b/Documentation/blame-options.txt\nindex 117f4cf806..b15999a3c8 100644\n--- a/Documentation/blame-options.txt\n+++ b/Documentation/blame-options.txt\n@@ -136,5 +136,13 @@ take effect.\n \toption. An empty file name, `\"\"`, will clear the list of revs from\n \tpreviously processed files.\n \n+--color-lines::\n+\tColor lines that contain repeated metadata. The color used is set with\n+\t`color.blame.repeatedLines` config option.\n+\n+--color-by-age::\n+\tColor metadata depending on age of the line. `color.blame.highlightRecent`\n+\tconfig option controls what color is used for each range of age.\n+\n -h::\n \tShow help message.\ndiff --git a/Documentation/git-blame.txt b/Documentation/git-blame.txt\nindex 3bf5d5d8b4..cfdbad030b 100644\n--- a/Documentation/git-blame.txt\n+++ b/Documentation/git-blame.txt\n@@ -11,8 +11,8 @@ SYNOPSIS\n 'git blame' [-c] [-b] [-l] [--root] [-t] [-f] [-n] [-s] [-e] [-p] [-w] [--incremental]\n \t [-L ] [-S ] [-M] [-C] [-C] [-C] [--since=]\n \t [--ignore-rev ] [--ignore-revs-file ]\n-\t [--progress] [--abbrev=] [ | --contents | --reverse ..]\n-\t [--] \n+\t [--color-lines] [--color-by-age] [--progress] [--abbrev=]\n+\t [ | --contents | --reverse ..] [--] \n \n DESCRIPTION\n -----------\n\nbase-commit: 99c99ed8259bf070cd8ae7b51a94904b7cf5c161\n-- \n2.25.1\n\n"},{"id":"437072","messageId":"355a8c33-2063-80be-ae88-515daefef8c4@gmail.com","threadId":"56586","inReplyTo":"20210925121817.1089897-1-bagasdotme@gmail.com","subject":"Re: [PATCH] blame: document --color-* options","fromName":"Bagas Sanjaya","fromEmail":"bagasdotme@gmail.com","sentAt":"2021-09-25T12:24:51Z","receivedAt":"2021-09-25T12:24:56Z","isPatch":true,"sender":{"key":"bagasdotme@gmail.com","avatar":"https://avatars.githubusercontent.com/u/40219486?v=4"},"body":"On 25/09/21 19.18, Bagas Sanjaya wrote:\n> Commit cdc2d5f11f1a (builtin/blame: dim uninteresting metadata lines,\n> 2018-04-23) and 25d5f52901f0 (builtin/blame: highlight recently changed\n> lines, 2018-04-23) introduce --color-lines and --color-by-age options to\n> git blame, respectively. While both options are mentioned in usage help,\n> they aren't documented in git-blame(1). Document them.\n> \n> Signed-off-by: Bagas Sanjaya \n\nOops, I forgot Reported-by trailer.\n\nReported-by: Dr. Matthias St. Pierre \n\n-- \nAn old man doll... just what I always wanted! - Clara\n"},{"id":"437176","messageId":"xmqq5yulregh.fsf@gitster.g","threadId":"56586","inReplyTo":"20210925121817.1089897-1-bagasdotme@gmail.com","subject":"Re: [PATCH] blame: document --color-* options","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2021-09-27T19:44:14Z","receivedAt":"2021-09-27T19:44:20Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Bagas Sanjaya writes:\n\n> diff --git a/Documentation/blame-options.txt b/Documentation/blame-options.txt\n> index 117f4cf806..b15999a3c8 100644\n> --- a/Documentation/blame-options.txt\n> +++ b/Documentation/blame-options.txt\n> @@ -136,5 +136,13 @@ take effect.\n> \toption. An empty file name, `\"\"`, will clear the list of revs from\n> \tpreviously processed files.\n> \n> +--color-lines::\n> +\tColor lines that contain repeated metadata. The color used is set with\n> +\t`color.blame.repeatedLines` config option.\n> +\n> +--color-by-age::\n> +\tColor metadata depending on age of the line. `color.blame.highlightRecent`\n> +\tconfig option controls what color is used for each range of age.\n\nNowhere in \"git blame --help\" output we mention anything about\n\"metadata\", so the readers of this new description will be left\npuzzled what the word means in the context of this command.\n\nWe would need to pick words or phrases that readers can link easily\nwith the description of \"THE PORCELAIN FORMAT\" section.\n\nThanks.\n"},{"id":"437251","messageId":"fe78329d-07a7-bdf3-2bda-13def35f3de3@gmail.com","threadId":"56586","inReplyTo":"xmqq5yulregh.fsf@gitster.g","subject":"Re: [PATCH] blame: document --color-* options","fromName":"Bagas Sanjaya","fromEmail":"bagasdotme@gmail.com","sentAt":"2021-09-28T05:43:13Z","receivedAt":"2021-09-28T05:43:20Z","isPatch":true,"sender":{"key":"bagasdotme@gmail.com","avatar":"https://avatars.githubusercontent.com/u/40219486?v=4"},"body":"On 28/09/21 02.44, Junio C Hamano wrote:\n> Nowhere in \"git blame --help\" output we mention anything about\n> \"metadata\", so the readers of this new description will be left\n> puzzled what the word means in the context of this command.\n> \n> We would need to pick words or phrases that readers can link easily\n> with the description of \"THE PORCELAIN FORMAT\" section.\n\nIn git-blame(1), we only describe output format for \"THE PORCELAIN \nFORMAT\" as well as \"INCREMENTAL OUTPUT\", but not default format.\n\nOn the other hand, --color-* options only works on default format, not \nthe others.\n\n-- \nAn old man doll... just what I always wanted! - Clara\n"},{"id":"437283","messageId":"4078bebf2da14e5f8fc80a5ac7918151@ncp-e.com","threadId":"56586","inReplyTo":"fe78329d-07a7-bdf3-2bda-13def35f3de3@gmail.com","subject":"RE: [PATCH] blame: document --color-* options","fromName":"Dr. Matthias St. Pierre","fromEmail":"matthias.st.pierre@ncp-e.com","sentAt":"2021-09-28T12:38:42Z","receivedAt":"2021-09-28T12:39:02Z","isPatch":true,"sender":{"key":"matthias.st.pierre@ncp-e.com","avatar":"https://gravatar.com/avatar/b3123d9574916986285f6ac929edb3b776525afd0b4196e61e5126867de764c6?d=mp&s=160"},"body":"Thank you Bagas for adding the documentation. I think the only that disturbed Junio is the use of the word \"metadata\",\nwhich is used internally, but not in the official documentation aimed at the git user.\n\nStarting from your patch, I added some suggestions for alternative wordings.\n\nRegards,\nMatthias\n\n\n\ncommit ae2c59b7c76d9201d68aeb21b0ce57f2845732a1\nAuthor: Bagas Sanjaya \nDate: Tue Sep 28 10:11:23 2021 +0200\n\n blame: document --color-* options\n\n Commit cdc2d5f11f1a (builtin/blame: dim uninteresting metadata lines,\n 2018-04-23) and 25d5f52901f0 (builtin/blame: highlight recently changed\n lines, 2018-04-23) introduce --color-lines and --color-by-age options to\n git blame, respectively. While both options are mentioned in usage help,\n they aren't documented in git-blame(1). Document them.\n\n Co-authored-by: Dr. Matthias St. Pierre \n\ndiff --git a/Documentation/blame-options.txt b/Documentation/blame-options.txt\nindex 117f4cf806..1560f2b6df 100644\n--- a/Documentation/blame-options.txt\n+++ b/Documentation/blame-options.txt\n@@ -136,5 +136,15 @@ take effect.\n option. An empty file name, `\"\"`, will clear the list of revs from\n previously processed files.\n\n+--color-lines::\n+Color lines differently if they belong to the same commit as the preceding line.\n+This facilitates distinguishing code blocks introduced by different commits.\n+The color defaults to cyan and be adjusted using the `color.blame.repeatedLines`\n+config option.\n+\n+--color-by-age::\n+Color lines depending on the age of the line. The `color.blame.highlightRecent`\n+config option controls what color is used for which range of age.\n+\n -h::\n Show help message.\ndiff --git a/Documentation/config/color.txt b/Documentation/config/color.txt\nindex e05d520a86..a1cf36fc89 100644\n--- a/Documentation/config/color.txt\n+++ b/Documentation/config/color.txt\n@@ -9,8 +9,8 @@ color.advice.hint::\n Use customized color for hints.\n\n color.blame.highlightRecent::\n-This can be used to color the metadata of a blame line depending\n-on age of the line.\n+Used to color line annotations differently depending on the age of the commit\n+(`git blame --color-by-age`).\n +\n This setting should be set to a comma-separated list of color and date settings,\n starting and ending with a color, the dates should be set from oldest to newest.\n@@ -25,10 +25,9 @@ everything older than one year blue, recent changes between one month and\n one year old are kept white, and lines introduced within the last month are\n colored red.\n\n-color.blame.repeatedLines::\n-Use the customized color for the part of git-blame output that\n-is repeated meta information per line (such as commit id,\n-author name, date and timezone). Defaults to cyan.\n+color.blame.repeated\n+Use this color to colorize line annotations, if they belong to the same commit\n+as the preceding line (`git blame --color-lines`). Defaults to cyan.\n\n color.branch::\n A boolean to enable/disable color in the output of\ndiff --git a/Documentation/git-blame.txt b/Documentation/git-blame.txt\nindex 3bf5d5d8b4..cfdbad030b 100644\n--- a/Documentation/git-blame.txt\n+++ b/Documentation/git-blame.txt\n@@ -11,8 +11,8 @@ SYNOPSIS\n 'git blame' [-c] [-b] [-l] [--root] [-t] [-f] [-n] [-s] [-e] [-p] [-w] [--incremental]\n [-L ] [-S ] [-M] [-C] [-C] [-C] [--since=]\n [--ignore-rev ] [--ignore-revs-file ]\n- [--progress] [--abbrev=] [ | --contents | --reverse ..]\n- [--] \n+ [--color-lines] [--color-by-age] [--progress] [--abbrev=]\n+ [ | --contents | --reverse ..] [--] \n\n DESCRIPTION\n -----------\n\n\nDr. Matthias St. Pierre\nTech Lead Cryptography\nmatthias.st.pierre@ncp-e.com\nPhone: +49 911 9968-0\n www.ncp-e.com\n\nHeadquarters Germany: NCP engineering GmbH • Dombuehler Str. 2 • 90449 • Nuremberg\nNorth American HQ: NCP engineering Inc. • 601 Cleveland Str., Suite 501-25 • Clearwater, FL 33755\n\nAuthorized representatives: Peter Soell, Patrick Oliver Graf, Beate Dietrich\nRegistry Court: Lower District Court of Nuremberg\nCommercial register No.: HRB 7786 Nuremberg, VAT identification No.: DE 133557619\n\nThis e-mail message including any attachments is for the sole use of the intended recipient(s) and may contain privileged\nor confidential information. Any unauthorized review, use, disclosure or distribution is prohibited. If you are not the intended recipient,\nplease immediately contact the sender by reply e-mail and delete the original message and destroy all copies thereof.\n"},{"id":"437326","messageId":"xmqq4ka4n00h.fsf@gitster.g","threadId":"56586","inReplyTo":"fe78329d-07a7-bdf3-2bda-13def35f3de3@gmail.com","subject":"Re: [PATCH] blame: document --color-* options","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2021-09-28T16:22:06Z","receivedAt":"2021-09-28T16:22:12Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Bagas Sanjaya writes:\n\n> On 28/09/21 02.44, Junio C Hamano wrote:\n>> Nowhere in \"git blame --help\" output we mention anything about\n>> \"metadata\", so the readers of this new description will be left\n>> puzzled what the word means in the context of this command.\n>> We would need to pick words or phrases that readers can link easily\n>> with the description of \"THE PORCELAIN FORMAT\" section.\n>\n> In git-blame(1), we only describe output format for \"THE PORCELAIN\n> FORMAT\" as well as \"INCREMENTAL OUTPUT\", but not default format.\n>\n> On the other hand, --color-* options only works on default format, not\n> the others.\n\nSorry for skipping a level of logical progression in the suggestion.\n\nWhile I do not think we want to add another section for the default\nformat, if we were to add such a section, the new description should\nuse phrases that would have been used, and such a section would not\nuse a new and unclear \"metadata\" in the description. Instead it\nwould use phrases that the users can look for elsewhere in the same\ndocumentation.\n\nWhat is colored is the summary of the commit the line came from\n(abbreviated commit object name, plus the author name and time by\ndefault) plus the line number, I think.\n\nThanks.\n"},{"id":"437441","messageId":"7d2fba09-4898-9b8d-d2c9-89438c8fb2e6@gmail.com","threadId":"56586","inReplyTo":"4078bebf2da14e5f8fc80a5ac7918151@ncp-e.com","subject":"Re: [PATCH] blame: document --color-* options","fromName":"Bagas Sanjaya","fromEmail":"bagasdotme@gmail.com","sentAt":"2021-09-29T08:01:35Z","receivedAt":"2021-09-29T08:02:09Z","isPatch":true,"sender":{"key":"bagasdotme@gmail.com","avatar":"https://avatars.githubusercontent.com/u/40219486?v=4"},"body":"On 28/09/21 19.38, Dr. Matthias St. Pierre wrote:\n> -color.blame.repeatedLines::\n> -Use the customized color for the part of git-blame output that\n> -is repeated meta information per line (such as commit id,\n> -author name, date and timezone). Defaults to cyan.\n> +color.blame.repeated\n> +Use this color to colorize line annotations, if they belong to the same commit\n> +as the preceding line (`git blame --color-lines`). Defaults to cyan.\n\nWhy did you change the config name? I think you make mistake here: the \nconfig name should stay as `color.blame.repeatedLines`.\n\n-- \nAn old man doll... just what I always wanted! - Clara\n"},{"id":"437446","messageId":"7745e157-7df9-2da7-f391-8c5bae16f005@gmail.com","threadId":"56586","inReplyTo":"xmqq4ka4n00h.fsf@gitster.g","subject":"Re: [PATCH] blame: document --color-* options","fromName":"Bagas Sanjaya","fromEmail":"bagasdotme@gmail.com","sentAt":"2021-09-29T09:15:55Z","receivedAt":"2021-09-29T09:16:01Z","isPatch":true,"sender":{"key":"bagasdotme@gmail.com","avatar":"https://avatars.githubusercontent.com/u/40219486?v=4"},"body":"On 28/09/21 23.22, Junio C Hamano wrote:\n> Sorry for skipping a level of logical progression in the suggestion.\n> \n> While I do not think we want to add another section for the default\n> format, if we were to add such a section, the new description should\n> use phrases that would have been used, and such a section would not\n> use a new and unclear \"metadata\" in the description. Instead it\n> would use phrases that the users can look for elsewhere in the same\n> documentation.\n> \n> What is colored is the summary of the commit the line came from\n> (abbreviated commit object name, plus the author name and time by\n> default) plus the line number, I think.\n> \n> Thanks.\n> \n\nStefan mentioned \"metadata\" term to describe the line annotation, from \ncommit cdc2d5f11f1a:\n\n> When using git-blame lots of lines contain redundant information, for\n> example in hunks that consist of multiple lines, the metadata (commit\n> name, author, date) are repeated. A reader may not be interested in those,\n> so offer an option to color the information that is repeated from the\n> previous line differently. Traditionally, we use CYAN for lines that\n> are less interesting than others (e.g. hunk header), so go with that.\n\n\n-- \nAn old man doll... just what I always wanted! - Clara\n"},{"id":"437447","messageId":"fa7df24dd20e4298883fb038689fff93@ncp-e.com","threadId":"56586","inReplyTo":"7d2fba09-4898-9b8d-d2c9-89438c8fb2e6@gmail.com","subject":"RE: [PATCH] blame: document --color-* options","fromName":"Dr. Matthias St. Pierre","fromEmail":"matthias.st.pierre@ncp-e.com","sentAt":"2021-09-29T09:19:06Z","receivedAt":"2021-09-29T09:19:21Z","isPatch":true,"sender":{"key":"matthias.st.pierre@ncp-e.com","avatar":"https://gravatar.com/avatar/b3123d9574916986285f6ac929edb3b776525afd0b4196e61e5126867de764c6?d=mp&s=160"},"body":"> On 28/09/21 19.38, Dr. Matthias St. Pierre wrote:\n> > -color.blame.repeatedLines::\n> > -Use the customized color for the part of git-blame output that\n> > -is repeated meta information per line (such as commit id,\n> > -author name, date and timezone). Defaults to cyan.\n> > +color.blame.repeated\n> > +Use this color to colorize line annotations, if they belong to the same commit\n> > +as the preceding line (`git blame --color-lines`). Defaults to cyan.\n>\n> Why did you change the config name? I think you make mistake here: the\n> config name should stay as `color.blame.repeatedLines`.\n\nYes, you are right. That was not an intended change, the name was truncated accidentally.\n\nMatthias\n\n\n\nDr. Matthias St. Pierre\nTech Lead Cryptography\nmatthias.st.pierre@ncp-e.com\nPhone: +49 911 9968-0\n www.ncp-e.com\n\nHeadquarters Germany: NCP engineering GmbH • Dombuehler Str. 2 • 90449 • Nuremberg\nNorth American HQ: NCP engineering Inc. • 601 Cleveland Str., Suite 501-25 • Clearwater, FL 33755\n\nAuthorized representatives: Peter Soell, Patrick Oliver Graf, Beate Dietrich\nRegistry Court: Lower District Court of Nuremberg\nCommercial register No.: HRB 7786 Nuremberg, VAT identification No.: DE 133557619\n\nThis e-mail message including any attachments is for the sole use of the intended recipient(s) and may contain privileged\nor confidential information. Any unauthorized review, use, disclosure or distribution is prohibited. If you are not the intended recipient,\nplease immediately contact the sender by reply e-mail and delete the original message and destroy all copies thereof.\n"},{"id":"437449","messageId":"c3603f8a-f75e-37f0-0517-c9a04c21e18e@gmail.com","threadId":"56586","inReplyTo":"4078bebf2da14e5f8fc80a5ac7918151@ncp-e.com","subject":"Re: [PATCH] blame: document --color-* options","fromName":"Bagas Sanjaya","fromEmail":"bagasdotme@gmail.com","sentAt":"2021-09-29T09:51:54Z","receivedAt":"2021-09-29T09:52:00Z","isPatch":true,"sender":{"key":"bagasdotme@gmail.com","avatar":"https://avatars.githubusercontent.com/u/40219486?v=4"},"body":"On 28/09/21 19.38, Dr. Matthias St. Pierre wrote:\n> Thank you Bagas for adding the documentation. I think the only that disturbed Junio is the use of the word \"metadata\",\n> which is used internally, but not in the official documentation aimed at the git user.\n> \n> Starting from your patch, I added some suggestions for alternative wordings.\n> \n> Regards,\n> Matthias\n> \n> \n> \n> commit ae2c59b7c76d9201d68aeb21b0ce57f2845732a1\n> Author: Bagas Sanjaya \n> Date: Tue Sep 28 10:11:23 2021 +0200\n> \n> blame: document --color-* options\n> \n> Commit cdc2d5f11f1a (builtin/blame: dim uninteresting metadata lines,\n> 2018-04-23) and 25d5f52901f0 (builtin/blame: highlight recently changed\n> lines, 2018-04-23) introduce --color-lines and --color-by-age options to\n> git blame, respectively. While both options are mentioned in usage help,\n> they aren't documented in git-blame(1). Document them.\n> \n> Co-authored-by: Dr. Matthias St. Pierre \n> \n> diff --git a/Documentation/blame-options.txt b/Documentation/blame-options.txt\n> index 117f4cf806..1560f2b6df 100644\n> --- a/Documentation/blame-options.txt\n> +++ b/Documentation/blame-options.txt\n> @@ -136,5 +136,15 @@ take effect.\n> option. An empty file name, `\"\"`, will clear the list of revs from\n> previously processed files.\n> \n> +--color-lines::\n> +Color lines differently if they belong to the same commit as the preceding line.\n> +This facilitates distinguishing code blocks introduced by different commits.\n> +The color defaults to cyan and be adjusted using the `color.blame.repeatedLines`\n> +config option.\n> +\n> +--color-by-age::\n> +Color lines depending on the age of the line. The `color.blame.highlightRecent`\n> +config option controls what color is used for which range of age.\n> +\n> -h::\n> Show help message.\n> diff --git a/Documentation/config/color.txt b/Documentation/config/color.txt\n> index e05d520a86..a1cf36fc89 100644\n> --- a/Documentation/config/color.txt\n> +++ b/Documentation/config/color.txt\n> @@ -9,8 +9,8 @@ color.advice.hint::\n> Use customized color for hints.\n> \n> color.blame.highlightRecent::\n> -This can be used to color the metadata of a blame line depending\n> -on age of the line.\n> +Used to color line annotations differently depending on the age of the commit\n> +(`git blame --color-by-age`).\n> +\n> This setting should be set to a comma-separated list of color and date settings,\n> starting and ending with a color, the dates should be set from oldest to newest.\n> @@ -25,10 +25,9 @@ everything older than one year blue, recent changes between one month and\n> one year old are kept white, and lines introduced within the last month are\n> colored red.\n> \n> -color.blame.repeatedLines::\n> -Use the customized color for the part of git-blame output that\n> -is repeated meta information per line (such as commit id,\n> -author name, date and timezone). Defaults to cyan.\n> +color.blame.repeated\n> +Use this color to colorize line annotations, if they belong to the same commit\n> +as the preceding line (`git blame --color-lines`). Defaults to cyan.\n> \n> color.branch::\n> A boolean to enable/disable color in the output of\n> diff --git a/Documentation/git-blame.txt b/Documentation/git-blame.txt\n> index 3bf5d5d8b4..cfdbad030b 100644\n> --- a/Documentation/git-blame.txt\n> +++ b/Documentation/git-blame.txt\n> @@ -11,8 +11,8 @@ SYNOPSIS\n> 'git blame' [-c] [-b] [-l] [--root] [-t] [-f] [-n] [-s] [-e] [-p] [-w] [--incremental]\n> [-L ] [-S ] [-M] [-C] [-C] [-C] [--since=]\n> [--ignore-rev ] [--ignore-revs-file ]\n> - [--progress] [--abbrev=] [ | --contents | --reverse ..]\n> - [--] \n> + [--color-lines] [--color-by-age] [--progress] [--abbrev=]\n> + [ | --contents | --reverse ..] [--] \n> \n> DESCRIPTION\n> -----------\n> \n> \n\nI can't apply the suggestion patch above. You sent the patch with \nS/MIME, right?\n\nNext time, whether you post patches here, learn to use git format-patch \nand git send-email. Send patches in plain text only, *no (S/)MIME, no \nlinks, no compression, no attachments*.\n\nPlease resend your suggestion patch using git send-email, keeping in \nmind the guidelines above.\n\n-- \nAn old man doll... just what I always wanted! - Clara\n"},{"id":"437450","messageId":"21b21b58e0da407387c9c1e103233a31@ncp-e.com","threadId":"56586","inReplyTo":"c3603f8a-f75e-37f0-0517-c9a04c21e18e@gmail.com","subject":"RE: [PATCH] blame: document --color-* options","fromName":"Dr. Matthias St. Pierre","fromEmail":"matthias.st.pierre@ncp-e.com","sentAt":"2021-09-29T10:53:07Z","receivedAt":"2021-09-29T10:53:11Z","isPatch":true,"sender":{"key":"matthias.st.pierre@ncp-e.com","avatar":"https://gravatar.com/avatar/b3123d9574916986285f6ac929edb3b776525afd0b4196e61e5126867de764c6?d=mp&s=160"},"body":"\n> I can't apply the suggestion patch above. You sent the patch with\n> S/MIME, right?\n>\n> Next time, whether you post patches here, learn to use git format-patch\n> and git send-email. Send patches in plain text only, *no (S/)MIME, no\n> links, no compression, no attachments*.\n>\n> Please resend your suggestion patch using git send-email, keeping in\n> mind the guidelines above.\n\nI'm sorry for the inconvenience: my company has a fancy-super-smart mail gateway\nwhich adds stuff to the mail which is out of my control. For that reason, git send-email\ncurrently doesn't work for me. So I kindly ask to pull the commit from my GitHub fork:\n\n git fetch https://github.com/mspncp/git.git document-git-blame-color-options\n git checkout -b document-git-blame-color-options\n\nThe truncated option name has already been corrected.\n\nMatthias\n\n\n\nDr. Matthias St. Pierre\nTech Lead Cryptography\nmatthias.st.pierre@ncp-e.com\nPhone: +49 911 9968-0\n www.ncp-e.com\n\nHeadquarters Germany: NCP engineering GmbH • Dombuehler Str. 2 • 90449 • Nuremberg\nNorth American HQ: NCP engineering Inc. • 601 Cleveland Str., Suite 501-25 • Clearwater, FL 33755\n\nAuthorized representatives: Peter Soell, Patrick Oliver Graf, Beate Dietrich\nRegistry Court: Lower District Court of Nuremberg\nCommercial register No.: HRB 7786 Nuremberg, VAT identification No.: DE 133557619\n\nThis e-mail message including any attachments is for the sole use of the intended recipient(s) and may contain privileged\nor confidential information. Any unauthorized review, use, disclosure or distribution is prohibited. If you are not the intended recipient,\nplease immediately contact the sender by reply e-mail and delete the original message and destroy all copies thereof.\n"},{"id":"437451","messageId":"1c7e7fbc0a06455f96b207c43cb37b99@ncp-e.com","threadId":"56586","inReplyTo":"21b21b58e0da407387c9c1e103233a31@ncp-e.com","subject":"RE: [PATCH] blame: document --color-* options","fromName":"Dr. Matthias St. Pierre","fromEmail":"matthias.st.pierre@ncp-e.com","sentAt":"2021-09-29T11:52:45Z","receivedAt":"2021-09-29T11:53:04Z","isPatch":true,"sender":{"key":"matthias.st.pierre@ncp-e.com","avatar":"https://gravatar.com/avatar/b3123d9574916986285f6ac929edb3b776525afd0b4196e61e5126867de764c6?d=mp&s=160"},"body":"Sorry, I forgot 'FETCH_HEAD':\n\n git fetch https://github.com/mspncp/git.git document-git-blame-color-options\n git checkout -b document-git-blame-color-options FETCH_HEAD\n\n\nDr. Matthias St. Pierre\nTech Lead Cryptography\nmatthias.st.pierre@ncp-e.com\nPhone: +49 911 9968-0\n www.ncp-e.com\n\nHeadquarters Germany: NCP engineering GmbH • Dombuehler Str. 2 • 90449 • Nuremberg\nNorth American HQ: NCP engineering Inc. • 601 Cleveland Str., Suite 501-25 • Clearwater, FL 33755\n\nAuthorized representatives: Peter Soell, Patrick Oliver Graf, Beate Dietrich\nRegistry Court: Lower District Court of Nuremberg\nCommercial register No.: HRB 7786 Nuremberg, VAT identification No.: DE 133557619\n\nThis e-mail message including any attachments is for the sole use of the intended recipient(s) and may contain privileged\nor confidential information. Any unauthorized review, use, disclosure or distribution is prohibited. If you are not the intended recipient,\nplease immediately contact the sender by reply e-mail and delete the original message and destroy all copies thereof.\n"},{"id":"437473","messageId":"xmqqk0izdyjj.fsf@gitster.g","threadId":"56586","inReplyTo":"7745e157-7df9-2da7-f391-8c5bae16f005@gmail.com","subject":"Re: [PATCH] blame: document --color-* options","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2021-09-29T18:30:56Z","receivedAt":"2021-09-29T18:31:00Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Bagas Sanjaya writes:\n\n> Stefan mentioned \"metadata\" term to describe the line annotation, from\n> commit cdc2d5f11f1a:\n>\n>> When using git-blame lots of lines contain redundant information, for\n>> example in hunks that consist of multiple lines, the metadata (commit\n>> name, author, date) are repeated. A reader may not be interested in those,\n>> so offer an option to color the information that is repeated from the\n>> previous line differently. Traditionally, we use CYAN for lines that\n>> are less interesting than others (e.g. hunk header), so go with that.\n\nThe documentation is for our end users, who may not have read our\ncommit log messages, so phrasing that was used in an old commit\nmessage would not help them.\n\n"}]}