{"thread":{"id":"39527","subject":"[RFC] send-email aliases when editing patches or using --xx-cmd","startedAt":"2015-06-04T20:17:51Z","lastAt":"2015-06-05T00:47:13Z","messageCount":7,"participants":["Remi LESPINET","Stefan Beller","Eric Sunshine","Remi Lespinet","Allen Hubbe"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"262977","messageId":"87pp5b1b4g.fsf@ensimag.grenoble-inp.fr","threadId":"39527","inReplyTo":null,"subject":"[RFC] send-email aliases when editing patches or using --xx-cmd","fromName":"Remi LESPINET","fromEmail":"remi.lespinet@ensimag.grenoble-inp.fr","sentAt":"2015-06-04T20:17:51Z","receivedAt":"2015-06-04T20:17:51Z","isPatch":false,"sender":{"key":"remi.lespinet@ensimag.grenoble-inp.fr","avatar":"https://avatars.githubusercontent.com/u/11941160?v=4"},"body":"\nHi,\n\nWorking on git-send-email, I've seen that there's no aliases support\nwhen manually adding a recipient in a 'To' or 'Cc' field in a patch\nand for the --to-cmd and --cc-cmd.\n\nI would like to add this support, and I wonder if there are reasons\nnot to do it.\n\nThanks.\n"},{"id":"262978","messageId":"CAGZ79ka89omT3wKqV-J4eyAz+xDXDS+OBcXDDFsO=4kk1HBBpQ@mail.gmail.com","threadId":"39527","inReplyTo":"87pp5b1b4g.fsf@ensimag.grenoble-inp.fr","subject":"Re: [RFC] send-email aliases when editing patches or using --xx-cmd","fromName":"Stefan Beller","fromEmail":"sbeller@google.com","sentAt":"2015-06-04T21:11:36Z","receivedAt":"2015-06-04T21:11:36Z","isPatch":false,"sender":{"key":"stefanbeller@gmail.com","avatar":"https://avatars.githubusercontent.com/u/455868?v=4"},"body":"On Thu, Jun 4, 2015 at 1:17 PM, Remi LESPINET\n wrote:\n>\n> Hi,\n>\n> Working on git-send-email, I've seen that there's no aliases support\n> when manually adding a recipient in a 'To' or 'Cc' field in a patch\n> and for the --to-cmd and --cc-cmd.\n>\n> I would like to add this support, and I wonder if there are reasons\n> not to do it.\n>\n> Thanks.\n\nJust recently Allen Hubbe did work on alias support,\nI did not follow that topic though.\n\n> --\n> To unsubscribe from this list: send the line \"unsubscribe git\" in\n> the body of a message to majordomo@vger.kernel.org\n> More majordomo info at http://vger.kernel.org/majordomo-info.html\n"},{"id":"262979","messageId":"CAPig+cQbOnaNg8dkbFoRrUN47oEAxUNjRXhMK1HSgCLi6ZxHyQ@mail.gmail.com","threadId":"39527","inReplyTo":"CAGZ79ka89omT3wKqV-J4eyAz+xDXDS+OBcXDDFsO=4kk1HBBpQ@mail.gmail.com","subject":"Re: [RFC] send-email aliases when editing patches or using --xx-cmd","fromName":"Eric Sunshine","fromEmail":"sunshine@sunshineco.com","sentAt":"2015-06-04T21:24:46Z","receivedAt":"2015-06-04T21:24:46Z","isPatch":false,"sender":{"key":"sunshine@sunshineco.com","avatar":"https://avatars.githubusercontent.com/u/163641?v=4"},"body":"On Thu, Jun 4, 2015 at 5:11 PM, Stefan Beller wrote:\n> On Thu, Jun 4, 2015 at 1:17 PM, Remi LESPINET\n> wrote:\n>> Working on git-send-email, I've seen that there's no aliases support\n>> when manually adding a recipient in a 'To' or 'Cc' field in a patch\n>> and for the --to-cmd and --cc-cmd.\n>>\n>> I would like to add this support, and I wonder if there are reasons\n>> not to do it.\n>\n> Just recently Allen Hubbe did work on alias support,\n> I did not follow that topic though.\n\nAllen's patch added support for another aliases format file but is\nunrelated to Remi's proposal and would not conflict with it.\n"},{"id":"262981","messageId":"1513043597.139303.1433455209117.JavaMail.zimbra@ensimag.grenoble-inp.fr","threadId":"39527","inReplyTo":"CAPig+cQbOnaNg8dkbFoRrUN47oEAxUNjRXhMK1HSgCLi6ZxHyQ@mail.gmail.com","subject":"[RFC] send-email aliases when editing patches or using --xx-cmd","fromName":"Remi Lespinet","fromEmail":"remi.lespinet@ensimag.grenoble-inp.fr","sentAt":"2015-06-04T22:00:09Z","receivedAt":"2015-06-04T22:00:09Z","isPatch":false,"sender":{"key":"remi.lespinet@ensimag.grenoble-inp.fr","avatar":"https://avatars.githubusercontent.com/u/11941160?v=4"},"body":"> On Thu, Jun 4, 2015 at 5:11 PM, Stefan Beller wrote:\n> > On Thu, Jun 4, 2015 at 1:17 PM, Remi LESPINET\n> > wrote:\n> >> Working on git-send-email, I've seen that there's no aliases support\n> >> when manually adding a recipient in a 'To' or 'Cc' field in a patch\n> >> and for the --to-cmd and --cc-cmd.\n> >>\n> >> I would like to add this support, and I wonder if there are reasons\n> >> not to do it.\n> >\n> > Just recently Allen Hubbe did work on alias support,\n> > I did not follow that topic though.\n> \n> Allen's patch added support for another aliases format file but is\n> unrelated to Remi's proposal and would not conflict with it.\n\nYes, I have read and applied the patches, this does not seems to be a\nproblem.\n\nThanks\n"},{"id":"262983","messageId":"299923407.139591.1433458411117.JavaMail.zimbra@ensimag.grenoble-inp.fr","threadId":"39527","inReplyTo":"CAPig+cQbOnaNg8dkbFoRrUN47oEAxUNjRXhMK1HSgCLi6ZxHyQ@mail.gmail.com","subject":"[RFC] send-email aliases when editing patches or using --xx-cmd","fromName":"Remi Lespinet","fromEmail":"remi.lespinet@ensimag.grenoble-inp.fr","sentAt":"2015-06-04T22:53:31Z","receivedAt":"2015-06-04T22:53:31Z","isPatch":false,"sender":{"key":"remi.lespinet@ensimag.grenoble-inp.fr","avatar":"https://avatars.githubusercontent.com/u/11941160?v=4"},"body":"On Thu, Jun 4, 2015 at 1:17 PM, Remi LESPINET\n wrote:\n>\n> Hi,\n>\n> Working on git-send-email, I've seen that there's no aliases support\n> when manually adding a recipient in a 'To' or 'Cc' field in a patch\n> and for the --to-cmd and --cc-cmd.\n>\n> I would like to add this support, and I wonder if there are reasons\n> not to do it.\n>\n> Thanks.\n\nI just realize that I totally messed up, Of course I don't want to add\nTo or Cc fields to patches.\n\nIn fact I want to add aliases support for --to-cmd and --cc-cmd options.\nBut the modification depends on wheter we can use aliases in fields\nused by send-email (such as author or signoff-by..) when manually\nediting a patch or not.\n\nSorry for this mistake :(\n"},{"id":"262988","messageId":"CAJ80savyruj7rcwGRY6AURJWnm26AyQFjssJmpjuYcv5jEBqkQ@mail.gmail.com","threadId":"39527","inReplyTo":"299923407.139591.1433458411117.JavaMail.zimbra@ensimag.grenoble-inp.fr","subject":"Re: [RFC] send-email aliases when editing patches or using --xx-cmd","fromName":"Allen Hubbe","fromEmail":"allenbh@gmail.com","sentAt":"2015-06-05T00:38:11Z","receivedAt":"2015-06-05T00:38:11Z","isPatch":false,"sender":{"key":"allenbh@gmail.com","avatar":"https://avatars.githubusercontent.com/u/812915?v=4"},"body":"On Thu, Jun 4, 2015 at 6:53 PM, Remi Lespinet\n wrote:\n> On Thu, Jun 4, 2015 at 1:17 PM, Remi LESPINET\n> wrote:\n>>\n>> Hi,\n>>\n>> Working on git-send-email, I've seen that there's no aliases support\n>> when manually adding a recipient in a 'To' or 'Cc' field in a patch\n>> and for the --to-cmd and --cc-cmd.\n>>\n>> I would like to add this support, and I wonder if there are reasons\n>> not to do it.\n>>\n>> Thanks.\n>\n> I just realize that I totally messed up, Of course I don't want to add\n> To or Cc fields to patches.\n>\n> In fact I want to add aliases support for --to-cmd and --cc-cmd options.\n> But the modification depends on wheter we can use aliases in fields\n> used by send-email (such as author or signoff-by..) when manually\n> editing a patch or not.\n>\n> Sorry for this mistake :(\n\nYou are right that Signed-off-by: myAlias wouldn't make sense, but is\nthat really what you intended to propose? It's definitely not a good\nidea to write patches that way, but on the other hand, if git happens\nto accept an alias there, what's the harm? Git will send an email...\nthe patch will be rejected.\n\nAlso, I believe git-send-email looks for signed-off-by, regardless of\nthe presence or output of to-cmd (around line 1490 -- \"Now parse the\nmessage body\").\n\nSomething like to-cmd is much more general than just looking for\nSigned-off-by, etc, or scanning a patch for things that look like\nemail addresses. The to-cmd can also look for keywords in the patch,\nlike component names, and return an array of maintainer email\naddresses for those components, for instance. It might be the case\nthat none of the emails returned by the to-cmd are actually ever\nexplicitly mentioned in the patch. In my workspace, I might indeed\nwrite a script, specify it as the to-cmd, and have the script return\nan array of email addresses according to whatever criteria I can\nimagine. What's the harm if, in my script, in my own workspace, I\nreturn an array of email aliases in the to_cmd, instead?\n\nNow, as long as the to-cmd is already a script, executed by the\ncomputer, there's really very little to gain in using email aliases\nversus whole email addresses. A script can emit a whole properly\nformatted email address just as easily as it can emit an email alias.\n\nYour proposal is just a different use for email aliases, which is\nindependent from where the aliases come from. There is no conflict\nwith any of the email alias parsers.\n"},{"id":"262990","messageId":"CAJ80sav_w9cFLkYAWUh-iGuV-9uG8c8gGjxF_V-QW=UA6ww4oQ@mail.gmail.com","threadId":"39527","inReplyTo":"CAJ80savyruj7rcwGRY6AURJWnm26AyQFjssJmpjuYcv5jEBqkQ@mail.gmail.com","subject":"Re: [RFC] send-email aliases when editing patches or using --xx-cmd","fromName":"Allen Hubbe","fromEmail":"allenbh@gmail.com","sentAt":"2015-06-05T00:47:13Z","receivedAt":"2015-06-05T00:47:13Z","isPatch":false,"sender":{"key":"allenbh@gmail.com","avatar":"https://avatars.githubusercontent.com/u/812915?v=4"},"body":"On Thu, Jun 4, 2015 at 8:38 PM, Allen Hubbe wrote:\n> On Thu, Jun 4, 2015 at 6:53 PM, Remi Lespinet\n> wrote:\n>> On Thu, Jun 4, 2015 at 1:17 PM, Remi LESPINET\n>> wrote:\n>>>\n>>> Hi,\n>>>\n>>> Working on git-send-email, I've seen that there's no aliases support\n>>> when manually adding a recipient in a 'To' or 'Cc' field in a patch\n>>> and for the --to-cmd and --cc-cmd.\n>>>\n>>> I would like to add this support, and I wonder if there are reasons\n>>> not to do it.\n>>>\n>>> Thanks.\n>>\n>> I just realize that I totally messed up, Of course I don't want to add\n>> To or Cc fields to patches.\n>>\n>> In fact I want to add aliases support for --to-cmd and --cc-cmd options.\n>> But the modification depends on wheter we can use aliases in fields\n>> used by send-email (such as author or signoff-by..) when manually\n>> editing a patch or not.\n>>\n>> Sorry for this mistake :(\n>\n> You are right that Signed-off-by: myAlias wouldn't make sense, but is\n> that really what you intended to propose? It's definitely not a good\n> idea to write patches that way, but on the other hand, if git happens\n> to accept an alias there, what's the harm? Git will send an email...\n> the patch will be rejected.\n>\n> Also, I believe git-send-email looks for signed-off-by, regardless of\n> the presence or output of to-cmd (around line 1490 -- \"Now parse the\n> message body\").\n>\n\nI don't see any harm to accepting aliases in the /header/ portion of\nthe patch, either. Those are not part of the commit message, and git\nwill reformat all the headers before actually sending the email,\nanyway.\n\nAgain, this is not the same as to-cmd. Note that git-send-email\nparses the header fields regardless of to-cmd.\n\n> Something like to-cmd is much more general than just looking for\n> Signed-off-by, etc, or scanning a patch for things that look like\n> email addresses. The to-cmd can also look for keywords in the patch,\n> like component names, and return an array of maintainer email\n> addresses for those components, for instance. It might be the case\n> that none of the emails returned by the to-cmd are actually ever\n> explicitly mentioned in the patch. In my workspace, I might indeed\n> write a script, specify it as the to-cmd, and have the script return\n> an array of email addresses according to whatever criteria I can\n> imagine. What's the harm if, in my script, in my own workspace, I\n> return an array of email aliases in the to_cmd, instead?\n>\n> Now, as long as the to-cmd is already a script, executed by the\n> computer, there's really very little to gain in using email aliases\n> versus whole email addresses. A script can emit a whole properly\n> formatted email address just as easily as it can emit an email alias.\n>\n> Your proposal is just a different use for email aliases, which is\n> independent from where the aliases come from. There is no conflict\n> with any of the email alias parsers.\n"}]}