Jan Ekström:
> On Sun, Sep 20, 2020 at 5:54 PM Andreas Rheinhardt
> <andreas.rheinha...@gmail.com> wrote:
>>
>> Otherwise the result of such tests will not accurately reflect the
>> current state.
>>
>> Signed-off-by: Andreas Rheinhardt <andreas.rheinha...@gmail.com>
>> ---
>> tests/fate-run.sh | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/tests/fate-run.sh b/tests/fate-run.sh
>> index 1b283e9c35..58d5fdbb60 100755
>> --- a/tests/fate-run.sh
>> +++ b/tests/fate-run.sh
>> @@ -158,7 +158,7 @@ md5pipe(){
>> md5(){
>> encfile="${outdir}/${test}.out"
>> cleanfiles="$cleanfiles $encfile"
>> - ffmpeg "$@" $(target_path $encfile)
>> + ffmpeg -y "$@" $(target_path $encfile)
>> do_md5sum $encfile | awk '{print $1}'
>> }
>>
>> --
>> 2.25.1
>
> LGTM.
>
> I think I hit this when working with the movenc changes recently when
> I had to update fate-mov-gpmf-remux.
>
> In theory these tests should be using stdout + file redirection, but
> alas this is what's configured here at the moment :) .
>
> Jan
There is already a md5pipe test method for this (it uses our md5
protocol); the introduction of md5 test here was specifically done in
7ed6f9168b7fd13915e19b5cb1ed599c1a2d6d31 because the md5 protocol is not
seekable, so that the test results e.g. dependeded on the size of the
AVIO buffers (which determined whether a seek within the buffer
succeeded or not).
- Andreas
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
