{"thread":{"id":"2863","subject":"Why do we need [PATCH]?","startedAt":"2005-12-16T21:01:45Z","lastAt":"2005-12-16T21:42:31Z","messageCount":5,"participants":["Sam Ravnborg","Brian Gerst","Dave Jones","Junio C Hamano"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"13738","messageId":"20051216210145.GA25311@mars.ravnborg.org","threadId":"2863","inReplyTo":null,"subject":"Why do we need [PATCH]?","fromName":"Sam Ravnborg","fromEmail":"sam@ravnborg.org","sentAt":"2005-12-16T21:01:45Z","receivedAt":"2005-12-16T21:01:45Z","isPatch":true,"sender":{"key":"sam@ravnborg.org","avatar":"https://gravatar.com/avatar/168a912606ed0742d840bb365e3cc21db390c36531a58341dc7a069cc1f15f62?d=mp&s=160"},"body":"Hi git's.\n\nI've just applied a patch to my git tree using git-applymbox.\nIt worked like a charm as it often do.\nBut then when I did a quick check with gitk it stuck me that\nthe subject was prefixed with [PATCH].\nI am aware that the [PATCH] in subject tell me:\n\"patch is received from somewhere and applied by me to the .git tree\".\nThis is in comparsion to patches received when I merge a git tree for\nexample.\n\nBut with common practice to use sign-off I wonder the value of this\npatch marker.\nWhen browsing the kernel shortlog I often focus on first word in subject\n- cause this tells me what system/drivers is changed. But with the\n [PATCH] marker I have to read some non-sense to see actual subject.\n\nThe information is redundant since metadata already tell me who is the\nauthor and who committed the change.\nSo could we have it removed or if people continue to find it usefull\nthen at least hide it behind some option. Using the [PATCH] prefix is\nnot the natural thing to do with git.\n\nI looked at the source and found the -k option, but adding [PATCH]\nshould not be default behaviour so this is not the correct solution.\n\n\tSam\n"},{"id":"13741","messageId":"20051216211357.GL2821@redhat.com","threadId":"2863","inReplyTo":"20051216210145.GA25311@mars.ravnborg.org","subject":"Re: Why do we need [PATCH]?","fromName":"Dave Jones","fromEmail":"davej@redhat.com","sentAt":"2005-12-16T21:13:57Z","receivedAt":"2005-12-16T21:13:57Z","isPatch":true,"sender":{"key":"davej@redhat.com","avatar":null},"body":"On Fri, Dec 16, 2005 at 10:01:45PM +0100, Sam Ravnborg wrote:\n\n > I've just applied a patch to my git tree using git-applymbox.\n > It worked like a charm as it often do.\n > But then when I did a quick check with gitk it stuck me that\n > the subject was prefixed with [PATCH].\n > I am aware that the [PATCH] in subject tell me:\n > \"patch is received from somewhere and applied by me to the .git tree\".\n > This is in comparsion to patches received when I merge a git tree for\n > example.\n > \n > But with common practice to use sign-off I wonder the value of this\n > patch marker.\n > When browsing the kernel shortlog I often focus on first word in subject\n > - cause this tells me what system/drivers is changed. But with the\n > [PATCH] marker I have to read some non-sense to see actual subject.\n > \n > The information is redundant since metadata already tell me who is the\n > author and who committed the change.\n > So could we have it removed or if people continue to find it usefull\n > then at least hide it behind some option. Using the [PATCH] prefix is\n > not the natural thing to do with git.\n > \n > I looked at the source and found the -k option, but adding [PATCH]\n > should not be default behaviour so this is not the correct solution.\n\nI'm still using Linus' original 'applypatch' script, on which I think\nthis stuff was based, and this behaviour annoyed me too.\nSo I changed it to do this..\n\nif [ -f .treename ]; then\n TREENAME=`cat .treename`\nelse\n TREENAME='PATCH'\nfi\n(echo \"[$TREENAME] $SUBJECT\" ; if [ -s $MSGFILE ]; then echo ; cat $MSGFILE; fi ) > $final\n\n\nand then I have a .treename in say the AGP tree, containing just 'AGPGART',\nand the changelogs get the prefixes that actually mean something useful.\n(And those markers are invaluable when searching through history for changes)\n\n\t\tDave\n"},{"id":"13740","messageId":"43A32E33.6040709@didntduck.org","threadId":"2863","inReplyTo":"20051216210145.GA25311@mars.ravnborg.org","subject":"Re: Why do we need [PATCH]?","fromName":"Brian Gerst","fromEmail":"bgerst@didntduck.org","sentAt":"2005-12-16T21:14:27Z","receivedAt":"2005-12-16T21:14:27Z","isPatch":true,"sender":{"key":"bgerst@didntduck.org","avatar":null},"body":"Sam Ravnborg wrote:\n> Hi git's.\n> \n> I've just applied a patch to my git tree using git-applymbox.\n> It worked like a charm as it often do.\n> But then when I did a quick check with gitk it stuck me that\n> the subject was prefixed with [PATCH].\n> I am aware that the [PATCH] in subject tell me:\n> \"patch is received from somewhere and applied by me to the .git tree\".\n> This is in comparsion to patches received when I merge a git tree for\n> example.\n> \n> But with common practice to use sign-off I wonder the value of this\n> patch marker.\n> When browsing the kernel shortlog I often focus on first word in subject\n> - cause this tells me what system/drivers is changed. But with the\n> [PATCH] marker I have to read some non-sense to see actual subject.\n> \n> The information is redundant since metadata already tell me who is the\n> author and who committed the change.\n> So could we have it removed or if people continue to find it usefull\n> then at least hide it behind some option. Using the [PATCH] prefix is\n> not the natural thing to do with git.\n> \n> I looked at the source and found the -k option, but adding [PATCH]\n> should not be default behaviour so this is not the correct solution.\n> \n> \tSam\n\n[PATCH] makes sense for the actual email message so that the recipient \nknows that the message contains a patch. But I agree that it does not \nadd any value to the commit message in git. It actually detracts value, \nbecause it takes up limited space in the shortlog message.\n\n--\n\t\t\t\tBrian Gerst\n"},{"id":"13743","messageId":"7vpsnwenqy.fsf@assigned-by-dhcp.cox.net","threadId":"2863","inReplyTo":"20051216210145.GA25311@mars.ravnborg.org","subject":"Re: Why do we need [PATCH]?","fromName":"Junio C Hamano","fromEmail":"junkio@cox.net","sentAt":"2005-12-16T21:27:33Z","receivedAt":"2005-12-16T21:27:33Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Sam Ravnborg writes:\n\n> I looked at the source and found the -k option, but adding [PATCH]\n> should not be default behaviour so this is not the correct solution.\n\nThe \"correct\" solution depends on where you come from. That\nextra [PATCH] is a carryover from BK days, I was told by Linus,\nto make e-mailed things stand out --- I've never used BK but I\nam guessing that things were not as obvious as our commit\nmessages, perhaps? We have \"Author/Committer\" distinction so\nthe [PATCH] marker is redundant.\n\nThe rewrite, \"git-am\" does not bother with adding [PATCH], but\nthe original \"git-applymbox\", being everyday Linus' tool, was\nleft as it was, not to disrupt the workflow of Linus.\n"},{"id":"13747","messageId":"20051216214231.GA2903@mars.ravnborg.org","threadId":"2863","inReplyTo":"7vpsnwenqy.fsf@assigned-by-dhcp.cox.net","subject":"Re: Why do we need [PATCH]?","fromName":"Sam Ravnborg","fromEmail":"sam@ravnborg.org","sentAt":"2005-12-16T21:42:31Z","receivedAt":"2005-12-16T21:42:31Z","isPatch":true,"sender":{"key":"sam@ravnborg.org","avatar":"https://gravatar.com/avatar/168a912606ed0742d840bb365e3cc21db390c36531a58341dc7a069cc1f15f62?d=mp&s=160"},"body":"On Fri, Dec 16, 2005 at 01:27:33PM -0800, Junio C Hamano wrote:\n> Sam Ravnborg writes:\n> \n> > I looked at the source and found the -k option, but adding [PATCH]\n> > should not be default behaviour so this is not the correct solution.\n> \n> The \"correct\" solution depends on where you come from. That\n> extra [PATCH] is a carryover from BK days, I was told by Linus,\n> to make e-mailed things stand out --- I've never used BK but I\n> am guessing that things were not as obvious as our commit\n> messages, perhaps?\nYou can a typical bk commit here:\nhttp://linus.bkbits.net:8080/linux-2.5/cset@1.2243?nav=index.html|ChangeSet@-9M\n\nAnd here it made much more sense sine the author/comitter info are\nless structured. Also we do nto have the Signed-off-by: stuff back then.\n\n> We have \"Author/Committer\" distinction so\n> the [PATCH] marker is redundant.\n> \n> The rewrite, \"git-am\" does not bother with adding [PATCH], but\n> the original \"git-applymbox\", being everyday Linus' tool, was\n> left as it was, not to disrupt the workflow of Linus.\n\nThanks, I had forgotten the git-am rewrite.\nI will use that in the future - if I remember.\n\n\tSam\n"}]}