{"thread":{"id":"60816","subject":"[PATCH] gitk: add \"Hightlight commit name\" menu entry","startedAt":"2024-01-30T08:53:21Z","lastAt":"2024-03-25T18:56:12Z","messageCount":6,"participants":["Raphael Gallais-Pou","Raphaël Gallais-Pou","Marc Branchaud","Junio C Hamano"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"487596","messageId":"20240130085308.5440-1-rgallaispou@gmail.com","threadId":"60816","inReplyTo":null,"subject":"[PATCH] gitk: add \"Hightlight commit name\" menu entry","fromName":"Raphael Gallais-Pou","fromEmail":"rgallaispou@gmail.com","sentAt":"2024-01-30T08:53:08Z","receivedAt":"2024-01-30T08:53:21Z","isPatch":true,"sender":{"key":"rgallaispou@gmail.com","avatar":null},"body":"When working with diverged branches, some patches can appear several times\non different branches without having the need to merge those branches.\nOn the other hand you may have to port a specific patch on another\nbranch you are working on. The search with a SHA1 cannot be applied here\nsince they would differ.\n\nThis patch adds an entry in the main context menu to highlight every\ninstance of a commit.\n\nSigned-off-by: Raphael Gallais-Pou <rgallaispou@gmail.com>\n---\n gitk-git/gitk | 23 ++++++++++++++++++++---\n 1 file changed, 20 insertions(+), 3 deletions(-)\n\ndiff --git a/gitk-git/gitk b/gitk-git/gitk\nindex 7a087f123d..4b15230a16 100755\n--- a/gitk-git/gitk\n+++ b/gitk-git/gitk\n@@ -2672,6 +2672,7 @@ proc makewindow {} {\n         {mc \"Make patch\" command mkpatch}\n         {mc \"Create tag\" command mktag}\n         {mc \"Copy commit reference\" command copyreference}\n+\t{mc \"Highlight commit name\" command highlightcommitname}\n         {mc \"Write commit to file\" command writecommit}\n         {mc \"Create new branch\" command mkbranch}\n         {mc \"Cherry-pick this commit\" command cherrypick}\n@@ -9002,13 +9003,13 @@ proc rowmenu {x y id} {\n     if {$id ne $nullid && $id ne $nullid2} {\n         set menu $rowctxmenu\n         if {$mainhead ne {}} {\n-            $menu entryconfigure 8 -label [mc \"Reset %s branch to here\" $mainhead] -state normal\n+            $menu entryconfigure 9 -label [mc \"Reset %s branch to here\" $mainhead] -state normal\n         } else {\n-            $menu entryconfigure 8 -label [mc \"Detached head: can't reset\" $mainhead] -state disabled\n+            $menu entryconfigure 9 -label [mc \"Detached head: can't reset\" $mainhead] -state disabled\n         }\n-        $menu entryconfigure 10 -state $mstate\n         $menu entryconfigure 11 -state $mstate\n         $menu entryconfigure 12 -state $mstate\n+        $menu entryconfigure 13 -state $mstate\n     } else {\n         set menu $fakerowmenu\n     }\n@@ -9481,6 +9482,22 @@ proc copyreference {} {\n     clipboard append $reference\n }\n \n+proc highlightcommitname {} {\n+    global rowmenuid autosellen findstring gdttype\n+\n+    set format \"%s\"\n+    set cmd [list git show -s --pretty=format:$format --date=short]\n+    if {$autosellen < 40} {\n+        lappend cmd --abbrev=$autosellen\n+    }\n+    set reference [eval exec $cmd $rowmenuid]\n+    set findstring $reference\n+    set gdttype [mc \"containing:\"]\n+\n+    clipboard clear\n+    clipboard append $reference\n+}\n+\n proc writecommit {} {\n     global rowmenuid wrcomtop commitinfo wrcomcmd NS\n \n-- \n2.43.0\n\n"},{"id":"490951","messageId":"e47f9df1-be1f-45aa-9308-dfbc2124f29c@gmail.com","threadId":"60816","inReplyTo":"20240130085308.5440-1-rgallaispou@gmail.com","subject":"Re: [PATCH] gitk: add \"Hightlight commit name\" menu entry","fromName":"Raphaël Gallais-Pou","fromEmail":"rgallaispou@gmail.com","sentAt":"2024-03-19T19:45:49Z","receivedAt":"2024-03-19T19:45:53Z","isPatch":true,"sender":{"key":"rgallaispou@gmail.com","avatar":null},"body":"Hi,\n\nGentle ping since I haven't received any reviews or comment on this \npatch. :)\n\nDo you guys actually take patches for the gitk tool ? I feel like there \nis not much features added on this.\n\nRegards,\nRaphaël\n\nLe 30/01/2024 à 09:53, Raphael Gallais-Pou a écrit :\n> When working with diverged branches, some patches can appear several times\n> on different branches without having the need to merge those branches.\n> On the other hand you may have to port a specific patch on another\n> branch you are working on. The search with a SHA1 cannot be applied here\n> since they would differ.\n> \n> This patch adds an entry in the main context menu to highlight every\n> instance of a commit.\n> \n> Signed-off-by: Raphael Gallais-Pou <rgallaispou@gmail.com>\n> ---\n>   gitk-git/gitk | 23 ++++++++++++++++++++---\n>   1 file changed, 20 insertions(+), 3 deletions(-)\n> \n> diff --git a/gitk-git/gitk b/gitk-git/gitk\n> index 7a087f123d..4b15230a16 100755\n> --- a/gitk-git/gitk\n> +++ b/gitk-git/gitk\n> @@ -2672,6 +2672,7 @@ proc makewindow {} {\n>           {mc \"Make patch\" command mkpatch}\n>           {mc \"Create tag\" command mktag}\n>           {mc \"Copy commit reference\" command copyreference}\n> +\t{mc \"Highlight commit name\" command highlightcommitname}\n>           {mc \"Write commit to file\" command writecommit}\n>           {mc \"Create new branch\" command mkbranch}\n>           {mc \"Cherry-pick this commit\" command cherrypick}\n> @@ -9002,13 +9003,13 @@ proc rowmenu {x y id} {\n>       if {$id ne $nullid && $id ne $nullid2} {\n>           set menu $rowctxmenu\n>           if {$mainhead ne {}} {\n> -            $menu entryconfigure 8 -label [mc \"Reset %s branch to here\" $mainhead] -state normal\n> +            $menu entryconfigure 9 -label [mc \"Reset %s branch to here\" $mainhead] -state normal\n>           } else {\n> -            $menu entryconfigure 8 -label [mc \"Detached head: can't reset\" $mainhead] -state disabled\n> +            $menu entryconfigure 9 -label [mc \"Detached head: can't reset\" $mainhead] -state disabled\n>           }\n> -        $menu entryconfigure 10 -state $mstate\n>           $menu entryconfigure 11 -state $mstate\n>           $menu entryconfigure 12 -state $mstate\n> +        $menu entryconfigure 13 -state $mstate\n>       } else {\n>           set menu $fakerowmenu\n>       }\n> @@ -9481,6 +9482,22 @@ proc copyreference {} {\n>       clipboard append $reference\n>   }\n>   \n> +proc highlightcommitname {} {\n> +    global rowmenuid autosellen findstring gdttype\n> +\n> +    set format \"%s\"\n> +    set cmd [list git show -s --pretty=format:$format --date=short]\n> +    if {$autosellen < 40} {\n> +        lappend cmd --abbrev=$autosellen\n> +    }\n> +    set reference [eval exec $cmd $rowmenuid]\n> +    set findstring $reference\n> +    set gdttype [mc \"containing:\"]\n> +\n> +    clipboard clear\n> +    clipboard append $reference\n> +}\n> +\n>   proc writecommit {} {\n>       global rowmenuid wrcomtop commitinfo wrcomcmd NS\n>   \n"},{"id":"491154","messageId":"27b9c158-5cb1-46bf-851a-88a02448fa2d@xiplink.com","threadId":"60816","inReplyTo":"20240130085308.5440-1-rgallaispou@gmail.com","subject":"Re: [PATCH] gitk: add \"Hightlight commit name\" menu entry","fromName":"Marc Branchaud","fromEmail":"marcnarc@xiplink.com","sentAt":"2024-03-21T15:51:33Z","receivedAt":"2024-03-21T15:51:38Z","isPatch":true,"sender":{"key":"marcnarc@xiplink.com","avatar":"https://avatars.githubusercontent.com/u/14980203?v=4"},"body":"\nOn 2024-01-30 03:53, Raphael Gallais-Pou wrote:\n> When working with diverged branches, some patches can appear several times\n> on different branches without having the need to merge those branches.\n> On the other hand you may have to port a specific patch on another\n> branch you are working on. The search with a SHA1 cannot be applied here\n> since they would differ.\n> \n> This patch adds an entry in the main context menu to highlight every\n> instance of a commit.\n\nThanks for working on gitk!\n\nUnfortunately, I don't understand the description of your new option. \nHow is this different from the existing \"Find containing:\" feature? \nGitk can already highlights commits that match a specified string. \nPlease explain what gitk does when this new option is selected.\n\nAlso, please explain how your code identifies \"every instance\" of a \ncommit.  When I think of a \"commit instance\" I think of the\n\"git patch-id\" command, which I don't see here.\n\n(It looks to me like this is basically a shortcut to auto-fill gitk's \n\"containing:\" field with the subject line of the selected commit?)\n\n> Signed-off-by: Raphael Gallais-Pou <rgallaispou@gmail.com>\n> ---\n>   gitk-git/gitk | 23 ++++++++++++++++++++---\n>   1 file changed, 20 insertions(+), 3 deletions(-)\n> \n> diff --git a/gitk-git/gitk b/gitk-git/gitk\n> index 7a087f123d..4b15230a16 100755\n> --- a/gitk-git/gitk\n> +++ b/gitk-git/gitk\n> @@ -2672,6 +2672,7 @@ proc makewindow {} {\n>           {mc \"Make patch\" command mkpatch}\n>           {mc \"Create tag\" command mktag}\n>           {mc \"Copy commit reference\" command copyreference}\n> +\t{mc \"Highlight commit name\" command highlightcommitname}\n\nThis line is indented with a tab, but it should use spaces.\n\n>           {mc \"Write commit to file\" command writecommit}\n>           {mc \"Create new branch\" command mkbranch}\n>           {mc \"Cherry-pick this commit\" command cherrypick}\n> @@ -9002,13 +9003,13 @@ proc rowmenu {x y id} {\n>       if {$id ne $nullid && $id ne $nullid2} {\n>           set menu $rowctxmenu\n>           if {$mainhead ne {}} {\n> -            $menu entryconfigure 8 -label [mc \"Reset %s branch to here\" $mainhead] -state normal\n> +            $menu entryconfigure 9 -label [mc \"Reset %s branch to here\" $mainhead] -state normal\n>           } else {\n> -            $menu entryconfigure 8 -label [mc \"Detached head: can't reset\" $mainhead] -state disabled\n> +            $menu entryconfigure 9 -label [mc \"Detached head: can't reset\" $mainhead] -state disabled\n>           }\n> -        $menu entryconfigure 10 -state $mstate\n>           $menu entryconfigure 11 -state $mstate\n>           $menu entryconfigure 12 -state $mstate\n> +        $menu entryconfigure 13 -state $mstate\n>       } else {\n>           set menu $fakerowmenu\n>       }\n> @@ -9481,6 +9482,22 @@ proc copyreference {} {\n>       clipboard append $reference\n>   }\n>   \n> +proc highlightcommitname {} {\n> +    global rowmenuid autosellen findstring gdttype\n> +\n> +    set format \"%s\"\n> +    set cmd [list git show -s --pretty=format:$format --date=short]\n\nWhy bother with the $format variable here?  Couldn't you just quote the \n--pretty part?\n\t\"--pretty=format:%s\"\n(FYI, I am not a TCL/TK coder.)\n\n\t\tM.\n\n> +    if {$autosellen < 40} {\n> +        lappend cmd --abbrev=$autosellen\n> +    }\n> +    set reference [eval exec $cmd $rowmenuid]\n> +    set findstring $reference\n> +    set gdttype [mc \"containing:\"]\n> +\n> +    clipboard clear\n> +    clipboard append $reference\n> +}\n> +\n>   proc writecommit {} {\n>       global rowmenuid wrcomtop commitinfo wrcomcmd NS\n>   \n"},{"id":"491291","messageId":"146fbb87-9047-4d96-8d06-76f5710ca9e1@gmail.com","threadId":"60816","inReplyTo":"27b9c158-5cb1-46bf-851a-88a02448fa2d@xiplink.com","subject":"Re: [PATCH] gitk: add \"Hightlight commit name\" menu entry","fromName":"Raphaël Gallais-Pou","fromEmail":"rgallaispou@gmail.com","sentAt":"2024-03-23T09:45:14Z","receivedAt":"2024-03-23T09:45:18Z","isPatch":true,"sender":{"key":"rgallaispou@gmail.com","avatar":null},"body":"Hi Marc,\n\nLe 21/03/2024 à 16:51, Marc Branchaud a écrit :\n> \n> On 2024-01-30 03:53, Raphael Gallais-Pou wrote:\n>> When working with diverged branches, some patches can appear several \n>> times\n>> on different branches without having the need to merge those branches.\n>> On the other hand you may have to port a specific patch on another\n>> branch you are working on. The search with a SHA1 cannot be applied here\n>> since they would differ.\n>>\n>> This patch adds an entry in the main context menu to highlight every\n>> instance of a commit.\n> \n> Thanks for working on gitk!\n> \n> Unfortunately, I don't understand the description of your new option. \n> How is this different from the existing \"Find containing:\" feature? Gitk \n> can already highlights commits that match a specified string. Please \n> explain what gitk does when this new option is selected.\n> \n> Also, please explain how your code identifies \"every instance\" of a \n> commit.  When I think of a \"commit instance\" I think of the\n> \"git patch-id\" command, which I don't see here.\n\nIt is based on the name of the commit. I agree that it is not ideal \nsince the name can change between two versions.\n\nAs you stated below it is exactly a shortcut, and now that we are \ntalking about it I think this is not the right right approach to do what \nI want.\n\nI was not aware of the 'git patch-id' command, but this it clearly a \nbetter idea to base the search of the commit instances on it.\n\nOne thing that I wonder is that 'git patch-id' seems to be based on \nstandard input. This means that in order to highlight every instances of \na commit the algorithm would need to parse each and every patch and then \nproceed to hash them and compare to the one referenced.\n\nWouldn't it be a tad long to process ?\n\n> \n> (It looks to me like this is basically a shortcut to auto-fill gitk's \n> \"containing:\" field with the subject line of the selected commit?)\n> \n>> Signed-off-by: Raphael Gallais-Pou <rgallaispou@gmail.com>\n>> ---\n>>   gitk-git/gitk | 23 ++++++++++++++++++++---\n>>   1 file changed, 20 insertions(+), 3 deletions(-)\n>>\n>> diff --git a/gitk-git/gitk b/gitk-git/gitk\n>> index 7a087f123d..4b15230a16 100755\n>> --- a/gitk-git/gitk\n>> +++ b/gitk-git/gitk\n>> @@ -2672,6 +2672,7 @@ proc makewindow {} {\n>>           {mc \"Make patch\" command mkpatch}\n>>           {mc \"Create tag\" command mktag}\n>>           {mc \"Copy commit reference\" command copyreference}\n>> +    {mc \"Highlight commit name\" command highlightcommitname}\n> \n> This line is indented with a tab, but it should use spaces.\n\nI will change my setup to use spaces instead of tabs one this file.\n\n> \n>>           {mc \"Write commit to file\" command writecommit}\n>>           {mc \"Create new branch\" command mkbranch}\n>>           {mc \"Cherry-pick this commit\" command cherrypick}\n>> @@ -9002,13 +9003,13 @@ proc rowmenu {x y id} {\n>>       if {$id ne $nullid && $id ne $nullid2} {\n>>           set menu $rowctxmenu\n>>           if {$mainhead ne {}} {\n>> -            $menu entryconfigure 8 -label [mc \"Reset %s branch to \n>> here\" $mainhead] -state normal\n>> +            $menu entryconfigure 9 -label [mc \"Reset %s branch to \n>> here\" $mainhead] -state normal\n>>           } else {\n>> -            $menu entryconfigure 8 -label [mc \"Detached head: can't \n>> reset\" $mainhead] -state disabled\n>> +            $menu entryconfigure 9 -label [mc \"Detached head: can't \n>> reset\" $mainhead] -state disabled\n>>           }\n>> -        $menu entryconfigure 10 -state $mstate\n>>           $menu entryconfigure 11 -state $mstate\n>>           $menu entryconfigure 12 -state $mstate\n>> +        $menu entryconfigure 13 -state $mstate\n>>       } else {\n>>           set menu $fakerowmenu\n>>       }\n>> @@ -9481,6 +9482,22 @@ proc copyreference {} {\n>>       clipboard append $reference\n>>   }\n>> +proc highlightcommitname {} {\n>> +    global rowmenuid autosellen findstring gdttype\n>> +\n>> +    set format \"%s\"\n>> +    set cmd [list git show -s --pretty=format:$format --date=short]\n> \n> Why bother with the $format variable here?  Couldn't you just quote the \n> --pretty part?\n>      \"--pretty=format:%s\"\n> (FYI, I am not a TCL/TK coder.)\n\nI also am not a TCL developer. I pretty much duplicated the \ncopyreference{} procedure to get what I wanted.\n\nBest regards,\nRaphaël\n> \n>          M.\n> \n>> +    if {$autosellen < 40} {\n>> +        lappend cmd --abbrev=$autosellen\n>> +    }\n>> +    set reference [eval exec $cmd $rowmenuid]\n>> +    set findstring $reference\n>> +    set gdttype [mc \"containing:\"]\n>> +\n>> +    clipboard clear\n>> +    clipboard append $reference\n>> +}\n>> +\n>>   proc writecommit {} {\n>>       global rowmenuid wrcomtop commitinfo wrcomcmd NS\n"},{"id":"491448","messageId":"742ac151-74cd-46f8-8dfb-3dcdaa513314@xiplink.com","threadId":"60816","inReplyTo":"146fbb87-9047-4d96-8d06-76f5710ca9e1@gmail.com","subject":"Re: [PATCH] gitk: add \"Hightlight commit name\" menu entry","fromName":"Marc Branchaud","fromEmail":"marcnarc@xiplink.com","sentAt":"2024-03-25T12:52:00Z","receivedAt":"2024-03-25T12:52:09Z","isPatch":true,"sender":{"key":"marcnarc@xiplink.com","avatar":"https://avatars.githubusercontent.com/u/14980203?v=4"},"body":"\nOn 2024-03-23 05:45, Raphaël Gallais-Pou wrote:\n> Hi Marc,\n> \n> Le 21/03/2024 à 16:51, Marc Branchaud a écrit :\n>>\n>> On 2024-01-30 03:53, Raphael Gallais-Pou wrote:\n>>> When working with diverged branches, some patches can appear several \n>>> times\n>>> on different branches without having the need to merge those branches.\n>>> On the other hand you may have to port a specific patch on another\n>>> branch you are working on. The search with a SHA1 cannot be applied here\n>>> since they would differ.\n>>>\n>>> This patch adds an entry in the main context menu to highlight every\n>>> instance of a commit.\n>>\n>> Thanks for working on gitk!\n>>\n>> Unfortunately, I don't understand the description of your new option. \n>> How is this different from the existing \"Find containing:\" feature? \n>> Gitk can already highlights commits that match a specified string. \n>> Please explain what gitk does when this new option is selected.\n>>\n>> Also, please explain how your code identifies \"every instance\" of a \n>> commit.  When I think of a \"commit instance\" I think of the\n>> \"git patch-id\" command, which I don't see here.\n> \n> It is based on the name of the commit. I agree that it is not ideal \n> since the name can change between two versions.\n\nThanks for clarifying that.\n\n(BTW, on this list we pronounce \"the name of the commit\" as \"the commit \nsubject (line)\".)\n\n> As you stated below it is exactly a shortcut, and now that we are \n> talking about it I think this is not the right right approach to do what \n> I want.\n> \n> I was not aware of the 'git patch-id' command, but this it clearly a \n> better idea to base the search of the commit instances on it.\n> \n> One thing that I wonder is that 'git patch-id' seems to be based on \n> standard input. This means that in order to highlight every instances of \n> a commit the algorithm would need to parse each and every patch and then \n> proceed to hash them and compare to the one referenced.\n> \n> Wouldn't it be a tad long to process ?\n\nAbsolutely.  Maybe \"git cherry\" would be more appropriate?\n\n\t\tM.\n\n\n>> (It looks to me like this is basically a shortcut to auto-fill gitk's \n>> \"containing:\" field with the subject line of the selected commit?)\n>>\n>>> Signed-off-by: Raphael Gallais-Pou <rgallaispou@gmail.com>\n>>> ---\n>>>   gitk-git/gitk | 23 ++++++++++++++++++++---\n>>>   1 file changed, 20 insertions(+), 3 deletions(-)\n>>>\n>>> diff --git a/gitk-git/gitk b/gitk-git/gitk\n>>> index 7a087f123d..4b15230a16 100755\n>>> --- a/gitk-git/gitk\n>>> +++ b/gitk-git/gitk\n>>> @@ -2672,6 +2672,7 @@ proc makewindow {} {\n>>>           {mc \"Make patch\" command mkpatch}\n>>>           {mc \"Create tag\" command mktag}\n>>>           {mc \"Copy commit reference\" command copyreference}\n>>> +    {mc \"Highlight commit name\" command highlightcommitname}\n>>\n>> This line is indented with a tab, but it should use spaces.\n> \n> I will change my setup to use spaces instead of tabs one this file.\n> \n>>\n>>>           {mc \"Write commit to file\" command writecommit}\n>>>           {mc \"Create new branch\" command mkbranch}\n>>>           {mc \"Cherry-pick this commit\" command cherrypick}\n>>> @@ -9002,13 +9003,13 @@ proc rowmenu {x y id} {\n>>>       if {$id ne $nullid && $id ne $nullid2} {\n>>>           set menu $rowctxmenu\n>>>           if {$mainhead ne {}} {\n>>> -            $menu entryconfigure 8 -label [mc \"Reset %s branch to \n>>> here\" $mainhead] -state normal\n>>> +            $menu entryconfigure 9 -label [mc \"Reset %s branch to \n>>> here\" $mainhead] -state normal\n>>>           } else {\n>>> -            $menu entryconfigure 8 -label [mc \"Detached head: can't \n>>> reset\" $mainhead] -state disabled\n>>> +            $menu entryconfigure 9 -label [mc \"Detached head: can't \n>>> reset\" $mainhead] -state disabled\n>>>           }\n>>> -        $menu entryconfigure 10 -state $mstate\n>>>           $menu entryconfigure 11 -state $mstate\n>>>           $menu entryconfigure 12 -state $mstate\n>>> +        $menu entryconfigure 13 -state $mstate\n>>>       } else {\n>>>           set menu $fakerowmenu\n>>>       }\n>>> @@ -9481,6 +9482,22 @@ proc copyreference {} {\n>>>       clipboard append $reference\n>>>   }\n>>> +proc highlightcommitname {} {\n>>> +    global rowmenuid autosellen findstring gdttype\n>>> +\n>>> +    set format \"%s\"\n>>> +    set cmd [list git show -s --pretty=format:$format --date=short]\n>>\n>> Why bother with the $format variable here?  Couldn't you just quote \n>> the --pretty part?\n>>      \"--pretty=format:%s\"\n>> (FYI, I am not a TCL/TK coder.)\n> \n> I also am not a TCL developer. I pretty much duplicated the \n> copyreference{} procedure to get what I wanted.\n> \n> Best regards,\n> Raphaël\n>>\n>>          M.\n>>\n>>> +    if {$autosellen < 40} {\n>>> +        lappend cmd --abbrev=$autosellen\n>>> +    }\n>>> +    set reference [eval exec $cmd $rowmenuid]\n>>> +    set findstring $reference\n>>> +    set gdttype [mc \"containing:\"]\n>>> +\n>>> +    clipboard clear\n>>> +    clipboard append $reference\n>>> +}\n>>> +\n>>>   proc writecommit {} {\n>>>       global rowmenuid wrcomtop commitinfo wrcomcmd NS\n"},{"id":"491489","messageId":"xmqq34sekunr.fsf@gitster.g","threadId":"60816","inReplyTo":"742ac151-74cd-46f8-8dfb-3dcdaa513314@xiplink.com","subject":"Re: [PATCH] gitk: add \"Hightlight commit name\" menu entry","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2024-03-25T18:56:08Z","receivedAt":"2024-03-25T18:56:12Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Marc Branchaud <marcnarc@xiplink.com> writes:\n\n> (BTW, on this list we pronounce \"the name of the commit\" as \"the\n> commit subject (line)\".)\n\nYup. \"subject\" probably comes from the fact that the title (the\nfirst paragraph, folded into a single line) of the commit is used on\nthe \"Subject:\" header when formatted for e-mail submission, and also\n's' in \"git log --pretty='%s'\" is described as \"subject\" in the\ndocumentation.  Other words I've seen used are \"commit title\" and\n\"oneline description\", but \"subject\" would be the most common, I\nsuspect.\n\n\"commit name\" on the other hand brings a different concept, i.e.,\n\"the commit object name\", more strongly to readers' minds, and can\nbe mistaken as what you'd get in \"git log --pretty='%H'\".\n"}]}