SZEDER Gábor <szeder....@gmail.com> writes:

>> +    git blame --ignore-rev $REV_2 --ignore-rev $REV_3 file | sed -e 
>> "$pick_author" >actual &&
>
> Please use an intermediate file between 'git blame' and the downstream
> 'sed' command, here and in other tests as well.
>
> This test script focuses specifically on 'git blame', but that pipe
> hides its exit code.

Good point on "this matters even more for 'blame' as the whole point
of t8014 _is_ about 'blame' running corretly".

Thanks.

Reply via email to