{"thread":{"id":"45918","subject":"[Query] Override sendemail.tocmd","startedAt":"2017-05-10T07:24:11Z","lastAt":"2017-05-18T09:41:05Z","messageCount":8,"participants":["Viresh Kumar","Ævar Arnfjörð Bjarmason"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"319259","messageId":"CAKohpok0=0__AcNZAfziTMYVDhGka3o0hzEN6HHZc=YUr7hWYQ@mail.gmail.com","threadId":"45918","inReplyTo":null,"subject":"[Query] Override sendemail.tocmd","fromName":"Viresh Kumar","fromEmail":"viresh.kumar@linaro.org","sentAt":"2017-05-10T07:23:59Z","receivedAt":"2017-05-10T07:24:11Z","isPatch":false,"sender":{"key":"viresh.kumar@linaro.org","avatar":"https://gravatar.com/avatar/938b1abce576d6c6d2734c01fce1c75040618c8fc0d1b7e836277ecd9f35fb19?d=mp&s=160"},"body":"Hi,\n\nI have a use case and wanted help from you guys..\n\nI have set the sendemail.tocmd option in my kernel's .git/config\nfile and that works very well. But sometimes I want to avoid that\n(for example sending the patch to someone without the list and\nmaintainers).\n\nThe suppress-cc=all option works quite well to avoid almost\neverything (including running --cc-cmd), but it doesn't suppress\n--to-cmd.\n\nAre there any specific reasons why it is being done that way?\n\nCurrently I need to go edit my .git/config file and remove tocmd\nbefore sending any such patches and it sucks :)\n\n--\nviresh\n"},{"id":"319260","messageId":"CAKohpokfVbE9=q6Rhpe0wQP9OuxyaN_z6sJaxdozGqqwTpinbA@mail.gmail.com","threadId":"45918","inReplyTo":"CAKohpok0=0__AcNZAfziTMYVDhGka3o0hzEN6HHZc=YUr7hWYQ@mail.gmail.com","subject":"Re: [Query] Override sendemail.tocmd","fromName":"Viresh Kumar","fromEmail":"viresh.kumar@linaro.org","sentAt":"2017-05-10T07:29:12Z","receivedAt":"2017-05-10T07:29:18Z","isPatch":false,"sender":{"key":"viresh.kumar@linaro.org","avatar":"https://gravatar.com/avatar/938b1abce576d6c6d2734c01fce1c75040618c8fc0d1b7e836277ecd9f35fb19?d=mp&s=160"},"body":"On 10 May 2017 at 12:53, Viresh Kumar wrote:\n> Hi,\n>\n> I have a use case and wanted help from you guys..\n>\n> I have set the sendemail.tocmd option in my kernel's .git/config\n> file and that works very well. But sometimes I want to avoid that\n> (for example sending the patch to someone without the list and\n> maintainers).\n>\n> The suppress-cc=all option works quite well to avoid almost\n> everything (including running --cc-cmd), but it doesn't suppress\n> --to-cmd.\n>\n> Are there any specific reasons why it is being done that way?\n>\n> Currently I need to go edit my .git/config file and remove tocmd\n> before sending any such patches and it sucks :)\n\nAnd it would be *really* useful if we can add the \"review\" (or whatever\nit is called) screen with suppress-cc command as well. Currently the mail\ngoes away directly without giving us a chance to look at the final list of\nrecipients the mail is going to :)\n\n--\nviresh\n"},{"id":"319265","messageId":"CACBZZX4pz26SCZBbivDneRX6hRPxpuPHXpOQKnn1zJL6xWj_Eg@mail.gmail.com","threadId":"45918","inReplyTo":"CAKohpok0=0__AcNZAfziTMYVDhGka3o0hzEN6HHZc=YUr7hWYQ@mail.gmail.com","subject":"Re: [Query] Override sendemail.tocmd","fromName":"Ævar Arnfjörð Bjarmason","fromEmail":"avarab@gmail.com","sentAt":"2017-05-10T10:18:02Z","receivedAt":"2017-05-10T10:18:28Z","isPatch":false,"sender":{"key":"avarab@gmail.com","avatar":"https://avatars.githubusercontent.com/u/45301?v=4"},"body":"On Wed, May 10, 2017 at 9:23 AM, Viresh Kumar wrote:\n> Hi,\n>\n> I have a use case and wanted help from you guys..\n>\n> I have set the sendemail.tocmd option in my kernel's .git/config\n> file and that works very well. But sometimes I want to avoid that\n> (for example sending the patch to someone without the list and\n> maintainers).\n>\n> The suppress-cc=all option works quite well to avoid almost\n> everything (including running --cc-cmd), but it doesn't suppress\n> --to-cmd.\n>\n> Are there any specific reasons why it is being done that way?\n>\n> Currently I need to go edit my .git/config file and remove tocmd\n> before sending any such patches and it sucks :)\n\nDoes something like this work:\n\n git -c sendemail.tocmd=/bin/false send-email ...\n\n?\n"},{"id":"319266","messageId":"20170510103400.GD17481@vireshk-i7","threadId":"45918","inReplyTo":"CACBZZX4pz26SCZBbivDneRX6hRPxpuPHXpOQKnn1zJL6xWj_Eg@mail.gmail.com","subject":"Re: [Query] Override sendemail.tocmd","fromName":"Viresh Kumar","fromEmail":"viresh.kumar@linaro.org","sentAt":"2017-05-10T10:34:01Z","receivedAt":"2017-05-10T10:34:08Z","isPatch":false,"sender":{"key":"viresh.kumar@linaro.org","avatar":"https://gravatar.com/avatar/938b1abce576d6c6d2734c01fce1c75040618c8fc0d1b7e836277ecd9f35fb19?d=mp&s=160"},"body":"On 10-05-17, 12:18, Ævar Arnfjörð Bjarmason wrote:\n> On Wed, May 10, 2017 at 9:23 AM, Viresh Kumar wrote:\n> > Hi,\n> >\n> > I have a use case and wanted help from you guys..\n> >\n> > I have set the sendemail.tocmd option in my kernel's .git/config\n> > file and that works very well. But sometimes I want to avoid that\n> > (for example sending the patch to someone without the list and\n> > maintainers).\n> >\n> > The suppress-cc=all option works quite well to avoid almost\n> > everything (including running --cc-cmd), but it doesn't suppress\n> > --to-cmd.\n> >\n> > Are there any specific reasons why it is being done that way?\n> >\n> > Currently I need to go edit my .git/config file and remove tocmd\n> > before sending any such patches and it sucks :)\n> \n> Does something like this work:\n> \n> git -c sendemail.tocmd=/bin/false send-email ...\n> \n> ?\n\nThat failed. I tried with 2.12 version first and then on the next branch with\nversion: 2.13.0.303.g4ebf3021692d\n\n$ git -c sendemail.tocmd=/bin/false send-email --to vingu 0001-abc.patch\n0001-abc.patch\n(mbox) Adding cc: Viresh Kumar from line 'From: Viresh Kumar '\n(body) Adding cc: Viresh Kumar from line 'Signed-off-by: Viresh Kumar '\n(to-cmd) failed to close pipe to '/bin/false' at /usr/libexec/git-core/git-send-email line 1687.\n\n-- \nviresh\n"},{"id":"319268","messageId":"CAKohpokJ6zUCUdOCp4LeaCjsmfAPTSr40MV94j2EbpE1tFXnFg@mail.gmail.com","threadId":"45918","inReplyTo":"CAKohpokfVbE9=q6Rhpe0wQP9OuxyaN_z6sJaxdozGqqwTpinbA@mail.gmail.com","subject":"Re: [Query] Override sendemail.tocmd","fromName":"Viresh Kumar","fromEmail":"viresh.kumar@linaro.org","sentAt":"2017-05-10T10:41:41Z","receivedAt":"2017-05-10T10:41:46Z","isPatch":false,"sender":{"key":"viresh.kumar@linaro.org","avatar":"https://gravatar.com/avatar/938b1abce576d6c6d2734c01fce1c75040618c8fc0d1b7e836277ecd9f35fb19?d=mp&s=160"},"body":"On 10 May 2017 at 12:59, Viresh Kumar wrote:\n> On 10 May 2017 at 12:53, Viresh Kumar wrote:\n>> Hi,\n>>\n>> I have a use case and wanted help from you guys..\n>>\n>> I have set the sendemail.tocmd option in my kernel's .git/config\n>> file and that works very well. But sometimes I want to avoid that\n>> (for example sending the patch to someone without the list and\n>> maintainers).\n>>\n>> The suppress-cc=all option works quite well to avoid almost\n>> everything (including running --cc-cmd), but it doesn't suppress\n>> --to-cmd.\n>>\n>> Are there any specific reasons why it is being done that way?\n>>\n>> Currently I need to go edit my .git/config file and remove tocmd\n>> before sending any such patches and it sucks :)\n>\n> And it would be *really* useful if we can add the \"review\" (or whatever\n> it is called) screen with suppress-cc command as well. Currently the mail\n> goes away directly without giving us a chance to look at the final list of\n> recipients the mail is going to :)\n\nFixed that with: sendemail.confirm = always\n"},{"id":"319269","messageId":"20170510104343.GE17481@vireshk-i7","threadId":"45918","inReplyTo":"20170510103400.GD17481@vireshk-i7","subject":"Re: [Query] Override sendemail.tocmd","fromName":"Viresh Kumar","fromEmail":"viresh.kumar@linaro.org","sentAt":"2017-05-10T10:43:43Z","receivedAt":"2017-05-10T10:43:52Z","isPatch":false,"sender":{"key":"viresh.kumar@linaro.org","avatar":"https://gravatar.com/avatar/938b1abce576d6c6d2734c01fce1c75040618c8fc0d1b7e836277ecd9f35fb19?d=mp&s=160"},"body":"On 10-05-17, 16:04, Viresh Kumar wrote:\n> On 10-05-17, 12:18, Ævar Arnfjörð Bjarmason wrote:\n> > On Wed, May 10, 2017 at 9:23 AM, Viresh Kumar wrote:\n> > > Hi,\n> > >\n> > > I have a use case and wanted help from you guys..\n> > >\n> > > I have set the sendemail.tocmd option in my kernel's .git/config\n> > > file and that works very well. But sometimes I want to avoid that\n> > > (for example sending the patch to someone without the list and\n> > > maintainers).\n> > >\n> > > The suppress-cc=all option works quite well to avoid almost\n> > > everything (including running --cc-cmd), but it doesn't suppress\n> > > --to-cmd.\n> > >\n> > > Are there any specific reasons why it is being done that way?\n> > >\n> > > Currently I need to go edit my .git/config file and remove tocmd\n> > > before sending any such patches and it sucks :)\n> > \n> > Does something like this work:\n> > \n> > git -c sendemail.tocmd=/bin/false send-email ...\n> > \n> > ?\n> \n> That failed. I tried with 2.12 version first and then on the next branch with\n> version: 2.13.0.303.g4ebf3021692d\n> \n> $ git -c sendemail.tocmd=/bin/false send-email --to vingu 0001-abc.patch\n> 0001-abc.patch\n> (mbox) Adding cc: Viresh Kumar from line 'From: Viresh Kumar '\n> (body) Adding cc: Viresh Kumar from line 'Signed-off-by: Viresh Kumar '\n> (to-cmd) failed to close pipe to '/bin/false' at /usr/libexec/git-core/git-send-email line 1687.\n\nAnd even if this worked, I believe that the right solution would be to fix\nsuppress-cc to include tocmd also. That's the whole purpose of suppress-cc\noption.\n\n-- \nviresh\n"},{"id":"319271","messageId":"CACBZZX6zVY=Nq9UZxK+ABtvfKF5Wi-QsGV4vRhg=S70x-YS3AQ@mail.gmail.com","threadId":"45918","inReplyTo":"20170510104343.GE17481@vireshk-i7","subject":"Re: [Query] Override sendemail.tocmd","fromName":"Ævar Arnfjörð Bjarmason","fromEmail":"avarab@gmail.com","sentAt":"2017-05-10T12:21:42Z","receivedAt":"2017-05-10T12:29:35Z","isPatch":false,"sender":{"key":"avarab@gmail.com","avatar":"https://avatars.githubusercontent.com/u/45301?v=4"},"body":"On Wed, May 10, 2017 at 12:43 PM, Viresh Kumar wrote:\n> On 10-05-17, 16:04, Viresh Kumar wrote:\n>> On 10-05-17, 12:18, Ćvar Arnfjörđ Bjarmason wrote:\n>> > On Wed, May 10, 2017 at 9:23 AM, Viresh Kumar wrote:\n>> > > Hi,\n>> > >\n>> > > I have a use case and wanted help from you guys..\n>> > >\n>> > > I have set the sendemail.tocmd option in my kernel's .git/config\n>> > > file and that works very well. But sometimes I want to avoid that\n>> > > (for example sending the patch to someone without the list and\n>> > > maintainers).\n>> > >\n>> > > The suppress-cc=all option works quite well to avoid almost\n>> > > everything (including running --cc-cmd), but it doesn't suppress\n>> > > --to-cmd.\n>> > >\n>> > > Are there any specific reasons why it is being done that way?\n>> > >\n>> > > Currently I need to go edit my .git/config file and remove tocmd\n>> > > before sending any such patches and it sucks :)\n>> >\n>> > Does something like this work:\n>> >\n>> > git -c sendemail.tocmd=/bin/false send-email ...\n>> >\n>> > ?\n>>\n>> That failed. I tried with 2.12 version first and then on the next branch with\n>> version: 2.13.0.303.g4ebf3021692d\n>>\n>> $ git -c sendemail.tocmd=/bin/false send-email --to vingu 0001-abc.patch\n>> 0001-abc.patch\n>> (mbox) Adding cc: Viresh Kumar from line 'From: Viresh Kumar '\n>> (body) Adding cc: Viresh Kumar from line 'Signed-off-by: Viresh Kumar '\n>> (to-cmd) failed to close pipe to '/bin/false' at /usr/libexec/git-core/git-send-email line 1687.\n>\n> And even if this worked, I believe that the right solution would be to fix\n> suppress-cc to include tocmd also. That's the whole purpose of suppress-cc\n> option.\n\nIndeed, it looks like either there should be a new option or the\nexisting option amended to work with tocmd as you suggest. I was just\nwondering if there was a way to do this now, even if with some\nnon-obvious hack.\n"},{"id":"320150","messageId":"CAKohpo=eFcf=VqCOvRJvQacUgMycZ1axvjbsNs2GeUGHzq4_Fg@mail.gmail.com","threadId":"45918","inReplyTo":"CAKohpok0=0__AcNZAfziTMYVDhGka3o0hzEN6HHZc=YUr7hWYQ@mail.gmail.com","subject":"Re: [Query] Override sendemail.tocmd","fromName":"Viresh Kumar","fromEmail":"viresh.kumar@linaro.org","sentAt":"2017-05-18T09:40:59Z","receivedAt":"2017-05-18T09:41:05Z","isPatch":false,"sender":{"key":"viresh.kumar@linaro.org","avatar":"https://gravatar.com/avatar/938b1abce576d6c6d2734c01fce1c75040618c8fc0d1b7e836277ecd9f35fb19?d=mp&s=160"},"body":"On 10 May 2017 at 12:53, Viresh Kumar wrote:\n> Hi,\n>\n> I have a use case and wanted help from you guys..\n>\n> I have set the sendemail.tocmd option in my kernel's .git/config\n> file and that works very well. But sometimes I want to avoid that\n> (for example sending the patch to someone without the list and\n> maintainers).\n>\n> The suppress-cc=all option works quite well to avoid almost\n> everything (including running --cc-cmd), but it doesn't suppress\n> --to-cmd.\n>\n> Are there any specific reasons why it is being done that way?\n>\n> Currently I need to go edit my .git/config file and remove tocmd\n> before sending any such patches and it sucks :)\n\nPing !!\n"}]}