[FFmpeg-devel] [PATCH 2/2] tests/fate/lavf-video.mak: better looking gif dependancies

James Almer jamrial at gmail.com
Mon Jan 6 16:43:29 EET 2020


On 1/6/2020 10:55 AM, Michael Niedermayer wrote:
> The gif test should depend on gif not fits
> 
> Signed-off-by: Michael Niedermayer <michael at niedermayer.cc>
> ---
>  tests/fate/lavf-video.mak | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/tests/fate/lavf-video.mak b/tests/fate/lavf-video.mak
> index 7a70ac90ce..f6e98246c4 100644
> --- a/tests/fate/lavf-video.mak
> +++ b/tests/fate/lavf-video.mak
> @@ -6,7 +6,7 @@ FATE_LAVF_VIDEO-$(call ENCDEC,  FITS,       FITS)               += gbrp.fits
>  FATE_LAVF_VIDEO-$(call ENCDEC,  FITS,       FITS)               += gbrap.fits
>  FATE_LAVF_VIDEO-$(call ENCDEC,  FITS,       FITS)               += gbrp16be.fits
>  FATE_LAVF_VIDEO-$(call ENCDEC,  FITS,       FITS)               += gbrap16be.fits
> -FATE_LAVF_VIDEO-$(call ENCDEC,  GIF,        FITS)               += gif
> +FATE_LAVF_VIDEO-$(call ENCDEC,  GIF,         GIF)               += gif

Before ac4b5d86222006fa71ffe5922e1a34f1422507d8 it was IMAGE2, not GIF
for the decoder part, so probably better to use that again.

>  FATE_LAVF_VIDEO-$(CONFIG_YUV4MPEGPIPE_MUXER)                    += y4m
>  
>  FATE_LAVF_VIDEO = $(FATE_LAVF_VIDEO-yes:%=fate-lavf-%)
> 

Make the commit message "fix fate-lavf-gif dependencies" instead of
"better looking gif dependancies". And mention the above commit as the
source of the regression.

LGTM otherwise.


More information about the ffmpeg-devel mailing list