diff mbox series

[FFmpeg-devel] fate/ffmpeg: add samples requirement to fate-ffmpeg-spec-disposition

Message ID 20241020203333.3133049-1-jeebjp@gmail.com
State New
Headers show
Series [FFmpeg-devel] fate/ffmpeg: add samples requirement to fate-ffmpeg-spec-disposition | expand

Commit Message

Jan Ekström Oct. 20, 2024, 8:33 p.m. UTC 
This tests utilizes an MPEG-TS sample from FATE suite, yet is
marked as not requiring samples.

This fixes running `make fate` without SAMPLES being set.

Reported-by: Sebastian Ramacher <sramacher@debian.org>
---
 tests/fate/ffmpeg.mak | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Jan Ekström Oct. 21, 2024, 6:26 p.m. UTC | #1 
On Sun, Oct 20, 2024 at 11:33 PM Jan Ekström <jeebjp@gmail.com> wrote:
>
> This tests utilizes an MPEG-TS sample from FATE suite, yet is
> marked as not requiring samples.
>
> This fixes running `make fate` without SAMPLES being set.
>
> Reported-by: Sebastian Ramacher <sramacher@debian.org>
> ---

Superceded by the earlier patch sent by Emily which is now applied to
both master and release/7.1.

Jan
diff mbox series

Patch

diff --git a/tests/fate/ffmpeg.mak b/tests/fate/ffmpeg.mak
index 869376dd18..d9348ee837 100644
--- a/tests/fate/ffmpeg.mak
+++ b/tests/fate/ffmpeg.mak
@@ -266,4 +266,4 @@  FATE_FFMPEG-$(call ENCDEC2, MPEG2VIDEO, FFV1, NUT, HSTACK_FILTER PIPE_PROTOCOL F
 
 # test matching by stream disposition
 fate-ffmpeg-spec-disposition: CMD = framecrc -i $(TARGET_SAMPLES)/mpegts/pmtchange.ts -map '0:disp:visual_impaired+descriptions:1' -c copy
-FATE_FFMPEG-$(call FRAMECRC, MPEGTS,,) += fate-ffmpeg-spec-disposition
+FATE_SAMPLES_FFMPEG-$(call FRAMECRC, MPEGTS,,) += fate-ffmpeg-spec-disposition