Hi Junio,
Thanks a lot for your review! (and many apologies for the double sending
as HTML…)
On 18/01/2024 21:16, Junio C Hamano wrote:
Or you could fix %O's description "while at it" and use the right
term from the get-go for %S.
Agreed, I'll do that, it also feels more fitting to me.
I see some funny indentation for "S" here.
Oops, sorry about that.
@@ -173,7 +179,7 @@ test_expect_success !WINDOWS 'custom merge driver that is killed with a signal'
git reset --hard anchor &&
git config --replace-all \
- merge.custom.driver "./custom-merge %O %A %B 0 %P" &&
+ merge.custom.driver "./custom-merge %O %A %B 0 %P %S %X %Y" &&
git config --replace-all \
merge.custom.name "custom merge driver for testing" &&
;-)
This one is expected to die and not produce meaningful output;
I was wondering why this does not need to make corresponding changes
to the expected output pattern like the earlier tests.
As far as I can tell, this test does not compare the result of the merge
to the expected merge driver output, because that output is expected to
be disregarded by git given that the merge driver died (see the last two
lines). So it seems normal to me that we don't need to adapt the
expected output: the repository files are still left unchanged.
I'll submit a new version of the patch with the two changes above.
Antonin
