[FFmpeg-devel] [PATCH] fate/matroska: Stop using unicode in command line paramters

Martin Storsjö martin at martin.st
Mon Aug 21 23:15:18 EEST 2023


The fate-run.sh shell script exports LC_ALL=C before invoking the
test executables; this is probably done for consistency.

When executing Windows binaries with Wine, it normally handles
UTF-8 command line parameters just fine - but with LC_ALL set to
C, it treats them as plain ASCII.

As the unicode command line parameters wasn't the main thing
being tested here, just convert them to plain ASCII, for
portability. This fixes the test for all test configurations that
use Wine.
---
 tests/fate/matroska.mak             |  6 +++---
 tests/ref/fate/matroska-stereo_mode | 10 +++++-----
 2 files changed, 8 insertions(+), 8 deletions(-)

diff --git a/tests/fate/matroska.mak b/tests/fate/matroska.mak
index 44f5c535e4..d1c316e998 100644
--- a/tests/fate/matroska.mak
+++ b/tests/fate/matroska.mak
@@ -228,10 +228,10 @@ FATE_MATROSKA_FFMPEG_FFPROBE-$(call REMUX, MATROSKA, OGG_DEMUXER THEORA_DECODER)
 fate-matroska-stereo_mode: CMD = transcode ogg $(TARGET_SAMPLES)/vp3/offset_test.ogv matroska \
     "-c copy -write_crc32 0 -default_mode infer \
      -map 0 -disposition:s:0 +original+dub -metadata:s:0 language=ger \
-     -map 0 -metadata:s:1 stereo_mode=left_right -metadata:s:1 language=ger-at -metadata:s:1 description-ger=Deutsch -metadata:s:1 description-fre=Français \
-     -map 0 -metadata:s:2 stereo_mode=bottom_top -metadata:s:2 language=eng -metadata:s:2 description-de=Deutsch -metadata:s:2 description-fra=Français \
+     -map 0 -metadata:s:1 stereo_mode=left_right -metadata:s:1 language=ger-at -metadata:s:1 description-ger=Deutsch -metadata:s:1 description-fre=Francais \
+     -map 0 -metadata:s:2 stereo_mode=bottom_top -metadata:s:2 language=eng -metadata:s:2 description-de=Deutsch -metadata:s:2 description-fra=Francais \
      -map 0 -metadata:s:3 stereo_mode=row_interleaved_rl -sar:3 3:1 -disposition:3 +default -metadata:s:3 language=deu-at \
-     -map 0 -metadata:s:4 stereo_mode=col_interleaved_rl -sar:4 16:9 -metadata:s:4 language=fre -metadata:s:4 description-deu-at=Österreichisch \
+     -map 0 -metadata:s:4 stereo_mode=col_interleaved_rl -sar:4 16:9 -metadata:s:4 language=fre -metadata:s:4 description-deu-at=Oesterreichisch \
      -map 0 -metadata:s:5 stereo_mode=anaglyph_cyan_red -sar:5 16:9 -disposition:5 +default -metadata:s:5 language=fra \
      -map 0 -metadata:s:6 stereo_mode=12 -sar:6 2:1 -metadata:s:6 language=de -metadata:s:6 description-deu=Deutsch" \
     "-map 0 -c copy" \
diff --git a/tests/ref/fate/matroska-stereo_mode b/tests/ref/fate/matroska-stereo_mode
index 3cd5e379f4..d406bb9b52 100644
--- a/tests/ref/fate/matroska-stereo_mode
+++ b/tests/ref/fate/matroska-stereo_mode
@@ -1,5 +1,5 @@
-1f074fcfb6f64ade0ad4b01f9e910515 *tests/data/fate/matroska-stereo_mode.matroska
-1470758 tests/data/fate/matroska-stereo_mode.matroska
+41cc6b0d334ac0465988579b99b78002 *tests/data/fate/matroska-stereo_mode.matroska
+1470756 tests/data/fate/matroska-stereo_mode.matroska
 #extradata 0:     3510, 0x560c3919
 #extradata 1:     3510, 0x560c3919
 #extradata 2:     3510, 0x560c3919
@@ -126,7 +126,7 @@ DISPOSITION:original=0
 TAG:language=ger-at
 TAG:stereo_mode=left_right
 TAG:DESCRIPTION-ger=Deutsch
-TAG:DESCRIPTION-fre=Français
+TAG:DESCRIPTION-fre=Francais
 TAG:DURATION=00:00:10.000000000
 [SIDE_DATA]
 side_data_type=Stereo 3D
@@ -141,7 +141,7 @@ DISPOSITION:original=0
 TAG:language=eng
 TAG:stereo_mode=bottom_top
 TAG:DESCRIPTION-ger=Deutsch
-TAG:DESCRIPTION-fre=Français
+TAG:DESCRIPTION-fre=Francais
 TAG:DURATION=00:00:10.000000000
 [SIDE_DATA]
 side_data_type=Stereo 3D
@@ -168,7 +168,7 @@ DISPOSITION:dub=0
 DISPOSITION:original=0
 TAG:language=fre
 TAG:stereo_mode=col_interleaved_rl
-TAG:DESCRIPTION-DEU-AT=Österreichisch
+TAG:DESCRIPTION-DEU-AT=Oesterreichisch
 TAG:DURATION=00:00:10.000000000
 [SIDE_DATA]
 side_data_type=Stereo 3D
-- 
2.34.1



More information about the ffmpeg-devel mailing list