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

From: Matthieu Moy <[email-masked]>
Date: Thu, 28 May 2015 13:47:21 +0200

Remi Lespinet <[email-masked]> 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/" \
> +		-e "s/^\(X-Mailer:\).*/\1 X-MAILER-STRING/"
> +}
> +
>  test_suppression () {
>  	git send-email \
>  		--dry-run \
> @@ -526,10 +532,7 @@ test_suppression () {
>  		--from="Example <[email-masked]>" \
>  		--to=[email-masked] \
>  		--smtp-server relay.example.com \
> -		$patches |
> -	sed	-e "s/^\(Date:\).*/\1 DATE-STRING/" \
> -		-e "s/^\(Message-Id:\).*/\1 MESSAGE-ID-STRING/" \
> -		-e "s/^\(X-Mailer:\).*/\1 X-MAILER-STRING/" \
> +		$patches | replace_variable_fields \

I wouldn't insist on that, but this change would be better done in a
separate, preparatory patch (that would be PATCH 1/2, and the actual
code would be PATCH 2/2).

-- 
Matthieu Moy
http://www-verimag.imag.fr/~moy/