Could someone take a look at this small fix, please? I'd like it to make it into 4.2.1.

On 2019-09-02 13:54, Andrey Semashev wrote:
Ping?

On 2019-08-28 18:32, Andrey Semashev wrote:
Because lavf_container is sometimes called with only 2 arguments,
fate tests produce bash errors like this:

   tests/fate-run.sh: 299: test: =: unexpected operator

This commit fixes this.
---
  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 2f1991da52..aec12c16a3 100755
--- a/tests/fate-run.sh
+++ b/tests/fate-run.sh
@@ -296,7 +296,7 @@ lavf_container(){
      outdir="tests/data/lavf"
      file=${outdir}/lavf.$t
      do_avconv $file $DEC_OPTS -f image2 -c:v pgmyuv -i $raw_src $DEC_OPTS -ar 44100 -f s16le $1 -i $pcm_src "$ENC_OPTS -metadata title=lavftest" -b:a 64k -t 1 -qscale:v 10 $2
-    test $3 = "disable_crc" ||
+    test "$3" = "disable_crc" ||
          do_avconv_crc $file $DEC_OPTS -i $target_path/$file $3
  }



_______________________________________________
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".

Reply via email to