On Tue, Mar 05, 2024 at 02:28:15PM -0800, Junio C Hamano wrote: > "Beat Bolli" <bb@xxxxxxxxx> writes: > > > Signed-off-by: Beat Bolli <dev+git@xxxxxxxxx> > > --- > > t/annotate-tests.sh | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/t/annotate-tests.sh b/t/annotate-tests.sh > > index 5e21e84f3884..87572459e4b8 100644 > > --- a/t/annotate-tests.sh > > +++ b/t/annotate-tests.sh > > @@ -532,7 +532,7 @@ test_expect_success 'blame -L :funcname with userdiff driver' ' > > "$(cat file.template)" && > > test_commit --author "B <B@xxxxxxxx>" \ > > "change" "$fortran_file" \ > > - "$(cat file.template | sed -e s/ChangeMe/IWasChanged/)" && > > + "$(sed -e s/ChangeMe/IWasChanged/ file.template)" && > > Obviously correct, but > > "$(sed -e s/ChangeMe/IWasChanged/ <file.template)" && > > might be a more faithful conversion (when "sed" looks at its ARGV[], > it did not find anything before, and it would not find anything > after this patch). Good point. Thank you for being careful. > > Not worth a reroll, of course, though.
