Re: [PATCH/RFC] send-email: allow multiple emails using --cc --to and --bcc

2015-05-29 Thread Junio C Hamano
Matthieu Moy writes: > Junio C Hamano writes: > >> Remi LESPINET writes: >> >>> Yes, that works if Foo is in an alias file, so that's clearly a bad >>> example, I added quotes: >>> >>> git send-email --to='"Foo, Bar" ' >> >> I'd further suggest replacing ", Bar" with something a bit more >>

Re: [PATCH/RFC] send-email: allow multiple emails using --cc --to and --bcc

2015-05-29 Thread Matthieu Moy
Junio C Hamano writes: > Remi LESPINET writes: > >> Yes, that works if Foo is in an alias file, so that's clearly a bad >> example, I added quotes: >> >> git send-email --to='"Foo, Bar" ' > > I'd further suggest replacing ", Bar" with something a bit more > realistic that people use in real

Re: [PATCH/RFC] send-email: allow multiple emails using --cc --to and --bcc

2015-05-29 Thread Junio C Hamano
Remi LESPINET writes: > Yes, that works if Foo is in an alias file, so that's clearly a bad > example, I added quotes: > > git send-email --to='"Foo, Bar" ' I'd further suggest replacing ", Bar" with something a bit more realistic that people use in real life, e.g. ", Esq." or ", PhD" (e.g

[PATCH/RFC] send-email: allow multiple emails using --cc --to and --bcc

2015-05-29 Thread Remi LESPINET
Eric Sunshine writes: > wrote: > > validate_address_list(sanitize_address_list( > split_address_list(@xx)) > > That's pretty verbose, so introducing a new function to encapsulates > that behavior might be reasonable. Agreed, If you have any suggestion for the name of this function

[PATCH/RFC] send-email: allow multiple emails using --cc --to and --bcc

2015-05-29 Thread Remi LESPINET
Junio C Hamano writes: > Remi Lespinet writes: > Accept a list of emails separated by commas in flags --cc, --to > and --bcc. Multiple addresses can already be given by using > these options multiple times, but it is more convenient to allow > cutting-and-pasting a list of addr

Re: [PATCH/RFC] send-email: allow multiple emails using --cc --to and --bcc

2015-05-28 Thread Eric Sunshine
On Thu, May 28, 2015 at 11:26 AM, Eric Sunshine wrote: > On Thu, May 28, 2015 at 6:42 AM, Remi Lespinet > wrote: >> + The format supported for email list is the following: >> + "Foo , b...@example.com". >> + Please notice that the email list does not handle commas in >> +

Re: [PATCH/RFC] send-email: allow multiple emails using --cc --to and --bcc

2015-05-28 Thread Junio C Hamano
Remi Lespinet writes: > From: Jorge Juan Garcia Garcia > > Add the possibility to use a list of emails separated by commas > in flags --cc --to and --bcc instead of having to use one flag > per email address. > > The use-case is to copy-paste a list of addresses from an email. > This change make

Re: [PATCH/RFC] send-email: allow multiple emails using --cc --to and --bcc

2015-05-28 Thread Eric Sunshine
On Thu, May 28, 2015 at 6:42 AM, Remi Lespinet wrote: > Add the possibility to use a list of emails separated by commas > in flags --cc --to and --bcc instead of having to use one flag s/--cc --to/--cc, --to/ Ditto in subject. > per email address. > > The use-case is to copy-paste a list of add

Re: [PATCH/RFC] send-email: allow multiple emails using --cc --to and --bcc

2015-05-28 Thread Matthieu Moy
Remi Lespinet writes: > --- a/t/t9001-send-email.sh > +++ b/t/t9001-send-email.sh > @@ -519,6 +519,12 @@ Result: OK > EOF > " > > +replace_variable_fields () { > + sed -e "s/^\(Date:\).*/\1 DATE-STRING/" \ > + -e "s/^\(Message-Id:\).*/\1 MESSAGE-ID-STRING/" \ > +

[PATCH/RFC] send-email: allow multiple emails using --cc --to and --bcc

2015-05-28 Thread Remi Lespinet
From: Jorge Juan Garcia Garcia Add the possibility to use a list of emails separated by commas in flags --cc --to and --bcc instead of having to use one flag per email address. The use-case is to copy-paste a list of addresses from an email. This change makes it so that we no longer need to cut

[PATCH/RFC] send-email: allow multiple emails using --cc --to and --bcc

2015-05-28 Thread Remi Lespinet
Hello, I've corrected an old patch from an Ensimag student. (http://thread.gmane.org/gmane.comp.version-control.git/228182). This patch allows multiple email addresses for options --cc, --to and --bcc. As said in the commit message, this patch doesn't handle commas in name, and the only possibilit