Remi Lespinet <[email protected]> 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 protected]>" \ > [email protected] \ > --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/ -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
