Matthieu Moy <[email protected]> writes: > Alex Bennée <[email protected]> writes: > >> I think you need to apply Eric's suggestions from: >> >> From: Eric Sunshine <[email protected]> >> Date: Sat, 18 Nov 2017 21:54:46 -0500 >> Message-ID: >> <capig+csh0tvvkh0xf9fwcfm4gngawmsn_fxd2zhzhcy2try...@mail.gmail.com> > > Indeed. I'm squashing this into the patch: > > diff --git a/t/t9001-send-email.sh b/t/t9001-send-email.sh > index f126177..d13d8c3 100755 > --- a/t/t9001-send-email.sh > +++ b/t/t9001-send-email.sh > @@ -173,8 +173,7 @@ test_expect_success $PREREQ 'cc trailer with various > syntax' ' > ' > > test_expect_success $PREREQ 'setup fake get_maintainer.pl script for cc > trailer' " > - cat >expected-cc-script.sh <<-EOF && > - #!/bin/sh > + write_script expected-cc-script.sh <<-EOF && > echo 'One Person <[email protected]> (supporter:THIS (FOO/bar))' > echo 'Two Person <[email protected]> (maintainer:THIS THING)' > echo 'Third List <[email protected]> (moderated list:THIS THING > (FOO/bar))'
Please do not forget to lose "chmod +x", which becomes unneeded when you use write_script. > @@ -186,7 +185,6 @@ test_expect_success $PREREQ 'setup fake get_maintainer.pl > script for cc trailer' > " > > test_expect_success $PREREQ 'cc trailer with get_maintainer.pl output' ' > - test_commit cc-trailer-getmaint && > clean_fake_sendmail && > git send-email -1 [email protected] \ > --cc-cmd="./expected-cc-script.sh" \
