# [PATCH] send-email uses contacts to propose recipients 4 messages from 2013-08-15 to 2013-08-15. Participants: Stefan Beller, Eric Sunshine, Junio C Hamano. Thread: https://gitlist.dev/t/34706 ## Stefan Beller, 2013-08-15 22:17 Subject: [PATCH] send-email uses contacts to propose recipients Message-ID: <1376605069-525-1-git-send-email-stefanbeller@googlemail.com> URL: https://gitlist.dev/e/1376605069-525-1-git-send-email-stefanbeller%40googlemail.com ``` I have got an idea regarding the send-email. If there are no recipients given, it could propose recipients using the new 'git contacts' This would help people new to projects to not forget people, who may have the most knowledge reviewing that specific patch. Unfortunately I cannot read/write perl, so I cannot solve it myself in the near future, but I'd put it on my todo list for later. --- git-send-email.perl | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/git-send-email.perl b/git-send-email.perl index 2162478..a34723d 100755 --- a/git-send-email.perl +++ b/git-send-email.perl @@ -757,6 +757,12 @@ $sender = sanitize_address($sender); my $prompting = 0; if (!@initial_to && !defined $to_cmd) { + if (git contacts is available) { + proposed senders = git contacts on the same range or set of patches + print "Suggesting these receivers:\n" + print proposed senders + } + my $to = ask("Who should the emails be sent to (if any)? ", default => "", valid_re => qr/\@.*\./, confirm_only => 1); -- 1.8.4.rc3.1.gc1ebd90 ``` ## Eric Sunshine, 2013-08-15 22:28 Subject: Re: [PATCH] send-email uses contacts to propose recipients Message-ID: URL: https://gitlist.dev/e/CAPig%2BcSZFU4oBuJCSXC%2B0Mgjegh4MzWCsDqdRYPLtwazwhy--Q%40mail.gmail.com In-Reply-To: <1376605069-525-1-git-send-email-stefanbeller@googlemail.com> ``` On Thu, Aug 15, 2013 at 6:17 PM, Stefan Beller wrote: > I have got an idea regarding the send-email. If there are no recipients > given, it could propose recipients using the new 'git contacts' > This would help people new to projects to not forget people, who may have > the most knowledge reviewing that specific patch. You can already do this with git-send-email's --cc-cmd option or sendmail.cccmd configuration variable. > Unfortunately I cannot read/write perl, so I cannot solve it > myself in the near future, but I'd put it on my todo list for later. > --- > git-send-email.perl | 6 ++++++ > 1 file changed, 6 insertions(+) > > diff --git a/git-send-email.perl b/git-send-email.perl > index 2162478..a34723d 100755 > --- a/git-send-email.perl > +++ b/git-send-email.perl > @@ -757,6 +757,12 @@ $sender = sanitize_address($sender); > > my $prompting = 0; > if (!@initial_to && !defined $to_cmd) { > + if (git contacts is available) { git-contacts is very slow, so at the very least you would want to prompt before invoking it. > + proposed senders = git contacts on the same range or set of patches > + print "Suggesting these receivers:\n" > + print proposed senders > + } > + > my $to = ask("Who should the emails be sent to (if any)? ", > default => "", > valid_re => qr/\@.*\./, confirm_only => 1); > -- > 1.8.4.rc3.1.gc1ebd90 > > -- > To unsubscribe from this list: send the line "unsubscribe git" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html ``` ## Junio C Hamano, 2013-08-15 22:29 Subject: Re: [PATCH] send-email uses contacts to propose recipients Message-ID: <7vhaeqpoyh.fsf@alter.siamese.dyndns.org> URL: https://gitlist.dev/e/7vhaeqpoyh.fsf%40alter.siamese.dyndns.org In-Reply-To: <1376605069-525-1-git-send-email-stefanbeller@googlemail.com> ``` Stefan Beller writes: > I have got an idea regarding the send-email. If there are no recipients > given, it could propose recipients using the new 'git contacts' > This would help people new to projects to not forget people, who may have > the most knowledge reviewing that specific patch. > > Unfortunately I cannot read/write perl, so I cannot solve it > myself in the near future, but I'd put it on my todo list for later. > --- > git-send-email.perl | 6 ++++++ > 1 file changed, 6 insertions(+) > > diff --git a/git-send-email.perl b/git-send-email.perl > index 2162478..a34723d 100755 > --- a/git-send-email.perl > +++ b/git-send-email.perl > @@ -757,6 +757,12 @@ $sender = sanitize_address($sender); > > my $prompting = 0; > if (!@initial_to && !defined $to_cmd) { > + if (git contacts is available) { > + proposed senders = git contacts on the same range or set of patches > + print "Suggesting these receivers:\n" > + print proposed senders > + } > + > my $to = ask("Who should the emails be sent to (if any)? ", > default => "", > valid_re => qr/\@.*\./, confirm_only => 1); Cute. It is OK while "contacts" is in contrib/, because people who has "git contact" are those who opted into the heuristics of implemented by that script. But if we are to eventually move the script out of the contrib/ area, we may have to restrict "is available" a bit tighter. Not everybody has to agree with its heuristics before the script moves out of the contrib/ area, so there will be users who are annoyed by the suggestion the script makes, which may not suit their needs at all. For contributors of _this_ list, the above will not kick in at all, as they should always have sendemail.to set to the list, and @initial_to will not be empty for them. ``` ## Junio C Hamano, 2013-08-15 22:38 Subject: Re: [PATCH] send-email uses contacts to propose recipients Message-ID: <7v4naqpojj.fsf@alter.siamese.dyndns.org> URL: https://gitlist.dev/e/7v4naqpojj.fsf%40alter.siamese.dyndns.org In-Reply-To: <7vhaeqpoyh.fsf@alter.siamese.dyndns.org> ``` Junio C Hamano writes: > ... we may have to restrict "is available" a bit > tighter. As Eric pointed out, that "bit tighter" opt-in could just be an explicit use of --cc-cmd option to specify this script ;-) ```