"Michael S. Tsirkin" <mst@xxxxxxxxxx> writes: > All I see in 2/6 is this: > > diff --git a/t/t9001-send-email.sh b/t/t9001-send-email.sh > index e1a7f3e..f81e5f5 100755 > --- a/t/t9001-send-email.sh > +++ b/t/t9001-send-email.sh > @@ -204,13 +204,15 @@ test_suppress_self_unquoted () { After noticing what is after "@@" here... > > unquoted-$3 > > + cccmd--$1 <$2> > + > Cc: $1 <$2> > Signed-off-by: $1 <$2> > EOF > } > > test_expect_success $PREREQ 'self name is suppressed' " > - test_suppress_self_unquoted 'A U Thor' 'author@xxxxxxxxxx' \ > + test_suppress_self_unquoted 'A U Thor' 'author@xxxxxxxxxxx' \ > 'self_name_suppressed' > " > > where's test_suppress_self_quoted here? ... isn't addition of "cccmd--$1 <$2>" made to that function? After applying [PATCH v3 1/6] to 'master', I do not see unquoted-$3 that we see in the context, either. "git grep unquoted- pu t/t9001*" does show us a hit, but that is coming from your previous round you are replacing with this series, so.... Confused... -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html