From: "Martin Storsjö" <martin@martin.st> To: ffmpeg-devel@ffmpeg.org Subject: [FFmpeg-devel] [PATCH] fate/matroska: Stop using unicode in command line paramters Date: Mon, 21 Aug 2023 23:15:18 +0300 Message-ID: <20230821201518.2132316-1-martin@martin.st> (raw) 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 _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".
next reply other threads:[~2023-08-21 20:15 UTC|newest] Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top 2023-08-21 20:15 Martin Storsjö [this message] 2023-08-22 19:08 ` Martin Storsjö
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20230821201518.2132316-1-martin@martin.st \ --to=martin@martin.st \ --cc=ffmpeg-devel@ffmpeg.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: link
Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel This inbox may be cloned and mirrored by anyone: git clone --mirror https://master.gitmailbox.com/ffmpegdev/0 ffmpegdev/git/0.git # If you have public-inbox 1.1+ installed, you may # initialize and index your mirror using the following commands: public-inbox-init -V2 ffmpegdev ffmpegdev/ https://master.gitmailbox.com/ffmpegdev \ ffmpegdev@gitmailbox.com public-inbox-index ffmpegdev Example config snippet for mirrors. AGPL code for this site: git clone https://public-inbox.org/public-inbox.git