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

2015-05-29 Thread Remi LESPINET
Eric Sunshine sunsh...@sunshineco.com writes: remi.lespi...@ensimag.grenoble-inp.fr 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

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

2015-05-29 Thread Junio C Hamano
Remi LESPINET remi.lespi...@ensimag.grenoble-inp.fr 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 foo...@example.com' I'd further suggest replacing , Bar with something a bit more realistic that people

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

2015-05-29 Thread Remi LESPINET
Junio C Hamano gits...@pobox.com writes: Remi Lespinet remi.lespi...@ensimag.grenoble-inp.fr 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

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

2015-05-29 Thread Matthieu Moy
Junio C Hamano gits...@pobox.com writes: Remi LESPINET remi.lespi...@ensimag.grenoble-inp.fr 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 foo...@example.com' I'd further suggest replacing , Bar with

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

2015-05-29 Thread Junio C Hamano
Matthieu Moy matthieu@grenoble-inp.fr writes: Junio C Hamano gits...@pobox.com writes: Remi LESPINET remi.lespi...@ensimag.grenoble-inp.fr 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

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

2015-05-28 Thread Junio C Hamano
Remi Lespinet remi.lespi...@ensimag.grenoble-inp.fr writes: From: Jorge Juan Garcia Garcia jorge-juan.garcia-gar...@ensimag.imag.fr 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

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 sunsh...@sunshineco.com wrote: On Thu, May 28, 2015 at 6:42 AM, Remi Lespinet remi.lespi...@ensimag.grenoble-inp.fr wrote: + The format supported for email list is the following: + Foo f...@example.com, b...@example.com. +

[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

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

2015-05-28 Thread Remi Lespinet
From: Jorge Juan Garcia Garcia jorge-juan.garcia-gar...@ensimag.imag.fr 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

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

2015-05-28 Thread Matthieu Moy
Remi Lespinet remi.lespi...@ensimag.grenoble-inp.fr 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

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 remi.lespi...@ensimag.grenoble-inp.fr 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