Jeremiah Mahler <jmmah...@gmail.com> writes:
> Below is one of the updated test cases.
> test_expect_success 'format-patch --signature-file=mail-signature' '
> cat >mail-signature <<-\EOF
> Test User <test.em...@kernel.org>
> git format-patch --stdout --signature-file=mail-signature -1 >output &&
> check_patch output &&
> sed -n -e "/^-- $/,\$p" <output | sed -e "1d" | sed -e "\$d" >output2 &&
> test_cmp mail-signature output2
Hmph, there are still few things I do not understand in the above.
* Why does mail-signature file have a leading blank line? Is it
typical users would want to have one there?
* Similarly, why does mail-signature file need a trailing blank
line? Is it usual users would want to have one there?
* Why three sed in the pipeline? Wouldn't
sed -e "1,/^-- \$/d" <output | ...
be more direct way to start the pipeline without having to strip
the "-- \n" line with the extra one?
* Given a mail-signature file that does not end with an incomplete
line (i.e. we did not have to add the newline to make it
complete), what are the expected lines in the output after the
"-- \n" separator?
Whatever it is, I think it is easier to read the tests if done
sed -e "1,/^-- \$/d" <output >actual &&
do something to turn mail-signature to what we expect
} >expect &&
test_cmp expect actual
If that "do something" is "to append an extra newline", then it
would make it a lot clear to do
cat mail-signature && echo
than a 'sed -e "\$d"' tucked at the end of the pipeline that only
tells us we are removing one line that has _something_ without
explicitly saying what we are removing, no?
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html