[FFmpeg-devel] [PATCH] fate/hlsend: rework the ffprobe dependency of hls-fmp4_ac3

Andreas Rheinhardt andreas.rheinhardt at gmail.com
Mon Jan 25 00:08:34 EET 2021


James Almer:
> Add it to the existing +FATE_SAMPLES_FFPROBE list of ffprobe dependant tests
> instead.
> 
> Signed-off-by: James Almer <jamrial at gmail.com>
> ---
>  tests/fate/hlsenc.mak | 7 ++++---
>  1 file changed, 4 insertions(+), 3 deletions(-)
> 
> diff --git a/tests/fate/hlsenc.mak b/tests/fate/hlsenc.mak
> index 02f041a7c2..30b81344f1 100644
> --- a/tests/fate/hlsenc.mak
> +++ b/tests/fate/hlsenc.mak
> @@ -105,9 +105,10 @@ tests/data/hls_fmp4_ac3.m3u8: ffmpeg$(PROGSSUF)$(EXESUF) | tests/data
>  	-hls_time 2 -hls_segment_filename "$(TARGET_PATH)/tests/data/hls_fmp4_ac3_%d.m4s" \
>  	$(TARGET_PATH)/tests/data/hls_fmp4_ac3.m3u8 2>/dev/null
>  
> -FATE_HLSENC-$(call ALLYES, HLS_DEMUXER EAC3_DEMUXER) += fate-hls-fmp4_ac3
> -fate-hls-fmp4_ac3: ffprobe$(PROGSSUF)$(EXESUF) tests/data/hls_fmp4_ac3.m3u8
> +FATE_HLSENC_PROBE-$(call ALLYES, HLS_DEMUXER EAC3_DEMUXER) += fate-hls-fmp4_ac3
> +fate-hls-fmp4_ac3: tests/data/hls_fmp4_ac3.m3u8
>  fate-hls-fmp4_ac3: CMD = probeaudiostream $(TARGET_PATH)/tests/data/now_ac3.mp4
>  
>  FATE_SAMPLES_FFMPEG += $(FATE_HLSENC-yes)
> -fate-hlsenc: $(FATE_HLSENC-yes)
> +FATE_SAMPLES_FFPROBE += $(FATE_HLSENC_PROBE-yes)
> +fate-hlsenc: $(FATE_HLSENC-yes) $(FATE_HLSENC_PROBE-yes)
> 
Why not FATE_SAMPLES_FFMPEG_FFPROBE? After all, it needs FFmpeg for the
input file.

- Andreas


More information about the ffmpeg-devel mailing list