{"thread":{"id":"48408","subject":"cover letter cc's [was: [PATCH 60/67] hw/s390x: add include directory headers]","startedAt":"2018-05-04T13:08:03Z","lastAt":"2018-05-04T21:38:55Z","messageCount":4,"participants":["Eric Blake","Cornelia Huck","Peter Maydell","Michael S. Tsirkin"],"isPatch":true,"patchVersion":1,"patchTotal":67},"messages":[{"id":"346646","messageId":"be79d6dd-beaa-ba6f-2990-b5e91139411b@redhat.com","threadId":"48408","inReplyTo":"20180504091007.55405a6a.cohuck@redhat.com","subject":"cover letter cc's [was: [PATCH 60/67] hw/s390x: add include directory headers]","fromName":"Eric Blake","fromEmail":"eblake@redhat.com","sentAt":"2018-05-04T13:07:53Z","receivedAt":"2018-05-04T13:08:03Z","isPatch":true,"sender":{"key":"eblake@redhat.com","avatar":"https://avatars.githubusercontent.com/u/32933908?v=4"},"body":"[adding a cross-post to the git mailing list]\n\nOn 05/04/2018 02:10 AM, Cornelia Huck wrote:\n> On Thu, 3 May 2018 22:51:40 +0300\n> \"Michael S. Tsirkin\" wrote:\n> \n>> This way they are easier to find using standard rules.\n>>\n>> Signed-off-by: Michael S. Tsirkin \n>> ---\n...\n\n> [Goes to find cover letter to figure out what this is all about.\n> *Please*, cc: people on the cover letter so they can see immediately\n> what this is trying to do!]\n\nIs there an EASY way to make 'git format-patch --cover-letter $commitid' \n(and git send-email, by extension) automatically search for all cc's any \nany of the N/M patches, and auto-cc ALL of those recipients on the 0/N \ncover letter? And if that is not something easily built into git \nformat-patch directly, is it something that can easily be added to \nsendemail.cccmd? This is not the first time that someone has complained \nthat automatic cc's are not sending the cover letter context to a \nparticular maintainer interested (and auto-cc'd) in only a subset of an \noverall series.\n\nOn the other hand, cc'ing all recipients for a largely mechanical patch \nseries that was split into 67 parts, in part because it touches so many \ndifferent maintainers' areas, may make the cover letter have so many \nrecipients that various mail gateways start rejecting it as potential spam.\n\n-- \nEric Blake, Principal Software Engineer\nRed Hat, Inc. +1-919-301-3266\nVirtualization: qemu.org | libvirt.org\n"},{"id":"346648","messageId":"20180504152658.3537707a.cohuck@redhat.com","threadId":"48408","inReplyTo":"be79d6dd-beaa-ba6f-2990-b5e91139411b@redhat.com","subject":"Re: cover letter cc's [was: [PATCH 60/67] hw/s390x: add include directory headers]","fromName":"Cornelia Huck","fromEmail":"cohuck@redhat.com","sentAt":"2018-05-04T13:26:58Z","receivedAt":"2018-05-04T13:27:11Z","isPatch":true,"sender":{"key":"cohuck@redhat.com","avatar":null},"body":"On Fri, 4 May 2018 08:07:53 -0500\nEric Blake wrote:\n\n> [adding a cross-post to the git mailing list]\n> \n> On 05/04/2018 02:10 AM, Cornelia Huck wrote:\n> > On Thu, 3 May 2018 22:51:40 +0300\n> > \"Michael S. Tsirkin\" wrote:\n> > \n> >> This way they are easier to find using standard rules.\n> >>\n> >> Signed-off-by: Michael S. Tsirkin \n> >> --- \n> ...\n> \n> > [Goes to find cover letter to figure out what this is all about.\n> > *Please*, cc: people on the cover letter so they can see immediately\n> > what this is trying to do!] \n> \n> Is there an EASY way to make 'git format-patch --cover-letter $commitid' \n> (and git send-email, by extension) automatically search for all cc's any \n> any of the N/M patches, and auto-cc ALL of those recipients on the 0/N \n> cover letter? And if that is not something easily built into git \n> format-patch directly, is it something that can easily be added to \n> sendemail.cccmd? This is not the first time that someone has complained \n> that automatic cc's are not sending the cover letter context to a \n> particular maintainer interested (and auto-cc'd) in only a subset of an \n> overall series.\n\nI think for most cases where I've been cc:ed on the cover letter and\nonly some of the patches, people actually added cc: lines to the cover\nletter manually.\n\n> \n> On the other hand, cc'ing all recipients for a largely mechanical patch \n> series that was split into 67 parts, in part because it touches so many \n> different maintainers' areas, may make the cover letter have so many \n> recipients that various mail gateways start rejecting it as potential spam.\n\nYes, large cross-subsystem patch series make this painful.\n\nIf I get some patches like \"subsystem: frobnicate foo\" and it's clear\nthat it's simply frobnicating foo for various subsystems, I can see\nwhat this is about without reading the cover letter, no need to cc: me.\nIn this case, however, the patch did not make any sense at all without\nlooking at the explanation in the cover letter.\n\nSo I think we don't want to do this automatically, although some way to\ncollect potential candidates for cc:ing on the cover letter might be\nhelpful.\n"},{"id":"346649","messageId":"CAFEAcA_xZZ5=oa-rA0=qh2-wxrt14KqH38rdg_OVLrqoTfzA9Q@mail.gmail.com","threadId":"48408","inReplyTo":"20180504152658.3537707a.cohuck@redhat.com","subject":"Re: cover letter cc's [was: [PATCH 60/67] hw/s390x: add include directory headers]","fromName":"Peter Maydell","fromEmail":"peter.maydell@linaro.org","sentAt":"2018-05-04T13:32:06Z","receivedAt":"2018-05-04T13:32:30Z","isPatch":true,"sender":{"key":"peter.maydell@linaro.org","avatar":null},"body":"On 4 May 2018 at 14:26, Cornelia Huck wrote:\n> On Fri, 4 May 2018 08:07:53 -0500\n> Eric Blake wrote:\n>> On the other hand, cc'ing all recipients for a largely mechanical patch\n>> series that was split into 67 parts, in part because it touches so many\n>> different maintainers' areas, may make the cover letter have so many\n>> recipients that various mail gateways start rejecting it as potential spam.\n>\n> Yes, large cross-subsystem patch series make this painful.\n\nMy solution is to either (a) avoid them or (b) not bother cc'ing\npeople (except people I think might be interested in reviewing\ncross-subsystem patchsets like Eric ;-))...\n\nthanks\n-- PMM\n"},{"id":"346694","messageId":"20180505003653-mutt-send-email-mst@kernel.org","threadId":"48408","inReplyTo":"be79d6dd-beaa-ba6f-2990-b5e91139411b@redhat.com","subject":"Re: cover letter cc's [was: [PATCH 60/67] hw/s390x: add include directory headers]","fromName":"Michael S. Tsirkin","fromEmail":"mst@redhat.com","sentAt":"2018-05-04T21:38:46Z","receivedAt":"2018-05-04T21:38:55Z","isPatch":true,"sender":{"key":"mst@kernel.org","avatar":null},"body":"On Fri, May 04, 2018 at 08:07:53AM -0500, Eric Blake wrote:\n> [adding a cross-post to the git mailing list]\n> \n> On 05/04/2018 02:10 AM, Cornelia Huck wrote:\n> > On Thu, 3 May 2018 22:51:40 +0300\n> > \"Michael S. Tsirkin\" wrote:\n> > \n> > > This way they are easier to find using standard rules.\n> > > \n> > > Signed-off-by: Michael S. Tsirkin \n> > > ---\n> ...\n> \n> > [Goes to find cover letter to figure out what this is all about.\n> > *Please*, cc: people on the cover letter so they can see immediately\n> > what this is trying to do!]\n> \n> Is there an EASY way to make 'git format-patch --cover-letter $commitid'\n> (and git send-email, by extension) automatically search for all cc's any any\n> of the N/M patches, and auto-cc ALL of those recipients on the 0/N cover\n> letter? And if that is not something easily built into git format-patch\n> directly, is it something that can easily be added to sendemail.cccmd? This\n> is not the first time that someone has complained that automatic cc's are\n> not sending the cover letter context to a particular maintainer interested\n> (and auto-cc'd) in only a subset of an overall series.\n> \n> On the other hand, cc'ing all recipients for a largely mechanical patch\n> series that was split into 67 parts, in part because it touches so many\n> different maintainers' areas, may make the cover letter have so many\n> recipients that various mail gateways start rejecting it as potential spam.\n\nI do this sometimes (pipe to this script):\n\ngrep -e ^Signed-off-by -e ^Acked -e ^Reported -e ^Tested -e ^Cc | sed 's/.*.*//'|sort | uniq | sed 's/^/Cc: /'\n\n\n> -- \n> Eric Blake, Principal Software Engineer\n> Red Hat, Inc. +1-919-301-3266\n> Virtualization: qemu.org | libvirt.org\n"}]}