{"thread":{"id":"3706","subject":"Re: Fw: [PATCH 31/49] PCI: PCI/Cardbus cards hidden, needs pci=assign-busses to fix","startedAt":"2006-03-24T00:46:54Z","lastAt":"2006-03-24T17:23:26Z","messageCount":8,"participants":["Greg KH","Junio C Hamano","Andrew Morton","Ryan Anderson"],"isPatch":true,"patchVersion":1,"patchTotal":49},"messages":[{"id":"17844","messageId":"20060324004654.GA19763@kroah.com","threadId":"3706","inReplyTo":"20060323163844.5fda7589.akpm@osdl.org","subject":"Re: Fw: [PATCH 31/49] PCI: PCI/Cardbus cards hidden, needs pci=assign-busses to fix","fromName":"Greg KH","fromEmail":"greg@kroah.com","sentAt":"2006-03-24T00:46:54Z","receivedAt":"2006-03-24T00:46:54Z","isPatch":true,"sender":{"key":"greg@kroah.com","avatar":"https://gravatar.com/avatar/5bb5aa0cc2e01c00ec899d11130c07796bc186e465bae57bc34873b13b72c7c8?d=mp&s=160"},"body":"On Thu, Mar 23, 2006 at 04:38:44PM -0800, Andrew Morton wrote:\n> \n> (added Junio)\n\nAdded the git mailing list as everyone should get involved :)\n\n> recap: the patch emails which Greg sends to linux-kernel do not identify\n> their actual Author.\n> \n> \n> Greg KH wrote:\n> >\n> > On Thu, Mar 23, 2006 at 04:15:21PM -0800, Andrew Morton wrote:\n> > > \n> > > It's unclear from this email who the patch author was?\n> > \n> > Git seems to strip that off when it converts them to emails. It was:\n> > \tFrom: Bernhard Kaindl \n> > \n> > If you look at the git changeset, it got it correct.\n> > \n> \n> OK. We really should have a\n> \n> \tFrom: Bernhard Kaindl \n> \n> right at the start of the email, IMO. Can you describe how you're producing\n> them please?\n\nI'm using:\n\tgit format-patch -n origin..HEAD\nto generate the raw patch files, and then:\n\tgit-send-email --in-reply-to \"\" --to some_mailing_list@somewhere.com\n\nfixing the obvious message id and mailing list address to be the correct\none depending on the subsystem the patches are from.\n\nthanks,\n\ngreg k-h\n"},{"id":"17845","messageId":"20060324005519.GA28344@kroah.com","threadId":"3706","inReplyTo":"20060324004654.GA19763@kroah.com","subject":"Re: Fw: [PATCH 31/49] PCI: PCI/Cardbus cards hidden, needs pci=assign-busses to fix","fromName":"Greg KH","fromEmail":"greg@kroah.com","sentAt":"2006-03-24T00:55:19Z","receivedAt":"2006-03-24T00:55:19Z","isPatch":true,"sender":{"key":"greg@kroah.com","avatar":"https://gravatar.com/avatar/5bb5aa0cc2e01c00ec899d11130c07796bc186e465bae57bc34873b13b72c7c8?d=mp&s=160"},"body":"On Thu, Mar 23, 2006 at 04:46:54PM -0800, Greg KH wrote:\n> On Thu, Mar 23, 2006 at 04:38:44PM -0800, Andrew Morton wrote:\n> > \n> > (added Junio)\n> \n> Added the git mailing list as everyone should get involved :)\n> \n> > recap: the patch emails which Greg sends to linux-kernel do not identify\n> > their actual Author.\n> > \n> > \n> > Greg KH wrote:\n> > >\n> > > On Thu, Mar 23, 2006 at 04:15:21PM -0800, Andrew Morton wrote:\n> > > > \n> > > > It's unclear from this email who the patch author was?\n> > > \n> > > Git seems to strip that off when it converts them to emails. It was:\n> > > \tFrom: Bernhard Kaindl \n> > > \n> > > If you look at the git changeset, it got it correct.\n> > > \n> > \n> > OK. We really should have a\n> > \n> > \tFrom: Bernhard Kaindl \n> > \n> > right at the start of the email, IMO. Can you describe how you're producing\n> > them please?\n> \n> I'm using:\n> \tgit format-patch -n origin..HEAD\n> to generate the raw patch files, and then:\n> \tgit-send-email --in-reply-to \"\" --to some_mailing_list@somewhere.com\n\nOops, forgot the \"*.txt\" at the end of that last line to specify the\npatches to send out.\n\nthanks,\n\ngreg k-h\n"},{"id":"17847","messageId":"7vbqvw3a62.fsf@assigned-by-dhcp.cox.net","threadId":"3706","inReplyTo":"20060324004654.GA19763@kroah.com","subject":"Re: Fw: [PATCH 31/49] PCI: PCI/Cardbus cards hidden, needs pci=assign-busses to fix","fromName":"Junio C Hamano","fromEmail":"junkio@cox.net","sentAt":"2006-03-24T01:26:29Z","receivedAt":"2006-03-24T01:26:29Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Greg KH writes:\n\n> I'm using:\n> \tgit format-patch -n origin..HEAD\n> to generate the raw patch files, and then:\n> \tgit-send-email --in-reply-to \"\" --to some_mailing_list@somewhere.com\n>\n> fixing the obvious message id and mailing list address to be the correct\n> one depending on the subsystem the patches are from.\n\nI think format-patch does the right thing (I wrote it), but I am\nnot sure what send-email does wrt the From: header. Who wrote\nthe send-email anyway? I see your name on it ;-)\n\nThe cleanest way send-email should handle a patch authored by\nsomebody other than you, I think, is to still use From: to name\nthe author (format-patch output records the author on From:\nline), and use Sender: of the outgoing e-mail to record that the\nmessage is from you. I suspect it probably doesn't.\n\nThe second best would be to add the duplicated From: to name the\nauthor (who is _not_ you) to the top of the body of the message.\nI do not particularly like that format myself, though. Sender:\nheader was invented to send an e-mail authored by somebody other\nthan the sender of the message at the mail transport level, long\nbefore Documentation/SubmittingPatches were written and git was\ninvented, and somehow I think that is a more kosher way to\nhandle that than the \"extra From: at the beginning of the\nmessage\" clutch recommended in SubmittingPatches document. \n\nOn the acceptance side, \"git am\" (or \"git applymbox\") should be\nable to handle either format.\n"},{"id":"17848","messageId":"20060323175126.7ff71032.akpm@osdl.org","threadId":"3706","inReplyTo":"7vbqvw3a62.fsf@assigned-by-dhcp.cox.net","subject":"Re: Fw: [PATCH 31/49] PCI: PCI/Cardbus cards hidden, needs pci=assign-busses to fix","fromName":"Andrew Morton","fromEmail":"akpm@osdl.org","sentAt":"2006-03-24T01:51:26Z","receivedAt":"2006-03-24T01:51:26Z","isPatch":true,"sender":{"key":"akpm@osdl.org","avatar":null},"body":"Junio C Hamano wrote:\n>\n> The second best would be to add the duplicated From: to name the\n> author (who is _not_ you) to the top of the body of the message.\n> I do not particularly like that format myself, though. Sender:\n> header was invented to send an e-mail authored by somebody other\n> than the sender of the message at the mail transport level, long\n> before Documentation/SubmittingPatches were written and git was\n> invented, and somehow I think that is a more kosher way to\n> handle that than the \"extra From: at the beginning of the\n> message\" clutch recommended in SubmittingPatches document. \n\nThe email I received from Greg had no Sender: header at all. I could find\nno indication of who authored the patch in that email.\n\nThe convention of adding the From: to the top of the body of the changelog\nis explicit and simple - I think it's a reasonable thing to do.\n\nWe wouldn't want to attempt to mix this concept up with email envelopes or\nemail headers or anything like that. The authorship is an attribute of the\npatch, and has nothing to do with how it was transported, stored or\nanything like that.\n"},{"id":"17849","messageId":"7v3bh837cs.fsf@assigned-by-dhcp.cox.net","threadId":"3706","inReplyTo":"20060323175126.7ff71032.akpm@osdl.org","subject":"Re: Fw: [PATCH 31/49] PCI: PCI/Cardbus cards hidden, needs pci=assign-busses to fix","fromName":"Junio C Hamano","fromEmail":"junkio@cox.net","sentAt":"2006-03-24T02:27:15Z","receivedAt":"2006-03-24T02:27:15Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Andrew Morton writes:\n\n> We wouldn't want to attempt to mix this concept up with email envelopes or\n> email headers or anything like that. The authorship is an attribute of the\n> patch, and has nothing to do with how it was transported, stored or\n> anything like that.\n\nFair enough. This is the approach I called \"the second best\" in\nmy message but I am inclined to agree with you.\n\nThis was tested once by sending myself two patches.\n\n-- >8 --\n[PATCH] send-email: Identify author at the top when sending e-mail\n\ngit-send-email was not checking if the sender is the same as the\npatch author. Follow the \"From: at the beginning\" convention to\npropagate the patch author correctly.\n\nSigned-off-by: Junio C Hamano \n---\ndiff --git a/git-send-email.perl b/git-send-email.perl\nindex 7c8d512..b220d11 100755\n--- a/git-send-email.perl\n+++ b/git-send-email.perl\n@@ -307,6 +307,7 @@ $subject = $initial_subject;\n foreach my $t (@files) {\n \topen(F,\"<\",$t) or die \"can't open file $t\";\n \n+\tmy $author_not_sender = undef;\n \t@cc = @initial_cc;\n \tmy $found_mbox = 0;\n \tmy $header_done = 0;\n@@ -321,7 +322,12 @@ foreach my $t (@files) {\n \t\t\t\t\t$subject = $1;\n \n \t\t\t\t} elsif (/^(Cc|From):\\s+(.*)$/) {\n-\t\t\t\t\tnext if ($2 eq $from && $suppress_from);\n+\t\t\t\t\tif ($2 eq $from) {\n+\t\t\t\t\t\tnext if ($suppress_from);\n+\t\t\t\t\t}\n+\t\t\t\t\telse {\n+\t\t\t\t\t\t$author_not_sender = $2;\n+\t\t\t\t\t}\n \t\t\t\t\tprintf(\"(mbox) Adding cc: %s from line '%s'\\n\",\n \t\t\t\t\t\t$2, $_) unless $quiet;\n \t\t\t\t\tpush @cc, $2;\n@@ -360,6 +366,9 @@ foreach my $t (@files) {\n \t\t}\n \t}\n \tclose F;\n+\tif (defined $author_not_sender) {\n+\t\t$message = \"From: $author_not_sender\\n\\n$message\";\n+\t}\n \n \t$cc = join(\", \", unique_email_list(@cc));\n \n"},{"id":"17852","messageId":"20060324061706.GA11248@kroah.com","threadId":"3706","inReplyTo":"7v3bh837cs.fsf@assigned-by-dhcp.cox.net","subject":"Re: Fw: [PATCH 31/49] PCI: PCI/Cardbus cards hidden, needs pci=assign-busses to fix","fromName":"Greg KH","fromEmail":"greg@kroah.com","sentAt":"2006-03-24T06:17:06Z","receivedAt":"2006-03-24T06:17:06Z","isPatch":true,"sender":{"key":"greg@kroah.com","avatar":"https://gravatar.com/avatar/5bb5aa0cc2e01c00ec899d11130c07796bc186e465bae57bc34873b13b72c7c8?d=mp&s=160"},"body":"On Thu, Mar 23, 2006 at 06:27:15PM -0800, Junio C Hamano wrote:\n> Andrew Morton writes:\n> \n> > We wouldn't want to attempt to mix this concept up with email envelopes or\n> > email headers or anything like that. The authorship is an attribute of the\n> > patch, and has nothing to do with how it was transported, stored or\n> > anything like that.\n> \n> Fair enough. This is the approach I called \"the second best\" in\n> my message but I am inclined to agree with you.\n> \n> This was tested once by sending myself two patches.\n\nOops, just saw this after I sent out the last set of patches. It looks\ngood to me, I'll try it out next time.\n\nAnd yes, I did write the original version of this perl script, but it's\nbeen fixed up and made useful by Ryan.\n\nthanks,\n\ngreg k-h\n"},{"id":"17853","messageId":"7vy7z01e3p.fsf@assigned-by-dhcp.cox.net","threadId":"3706","inReplyTo":"20060324061706.GA11248@kroah.com","subject":"Re: Fw: [PATCH 31/49] PCI: PCI/Cardbus cards hidden, needs pci=assign-busses to fix","fromName":"Junio C Hamano","fromEmail":"junkio@cox.net","sentAt":"2006-03-24T07:44:26Z","receivedAt":"2006-03-24T07:44:26Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Greg KH writes:\n\n>> This was tested once by sending myself two patches.\n>\n> Oops, just saw this after I sent out the last set of patches. It looks\n> good to me, I'll try it out next time.\n\nThanks. Will place this in \"master\" tonight.\n"},{"id":"17880","messageId":"20060324172326.GM26071@mythryan2.michonline.com","threadId":"3706","inReplyTo":"7v3bh837cs.fsf@assigned-by-dhcp.cox.net","subject":"Re: Fw: [PATCH 31/49] PCI: PCI/Cardbus cards hidden, needs pci=assign-busses to fix","fromName":"Ryan Anderson","fromEmail":"ryan@michonline.com","sentAt":"2006-03-24T17:23:26Z","receivedAt":"2006-03-24T17:23:26Z","isPatch":true,"sender":{"key":"ryan@michonline.com","avatar":null},"body":"On Thu, Mar 23, 2006 at 06:27:15PM -0800, Junio C Hamano wrote:\n> [PATCH] send-email: Identify author at the top when sending e-mail\n> \n> git-send-email was not checking if the sender is the same as the\n> patch author. Follow the \"From: at the beginning\" convention to\n> propagate the patch author correctly.\n\nThis looks fine.\nMy initial thought here was that it might be even simpler to just always\nput the From: line in the body, with the patch author's name there, but\nthis is a bit cleaner, so ACK.\n\n-- \n\nRyan Anderson\n sometimes Pug Majere\n"}]}