[FFmpeg-devel] [PATCH v6 3/4] configure: add FLIF dependencies

Anamitra Ghorui aghorui at teknik.io
Wed Aug 26 19:51:00 EEST 2020


On 26/08/2020 18:57, Moritz Barsnick wrote:
> On Sun, Aug 23, 2020 at 00:09:34 +0530, Anamitra Ghorui wrote:
>> --- a/configure
>> +++ b/configure
>> @@ -3307,6 +3307,7 @@ fifo_muxer_deps="threads"
>>   flac_demuxer_select="flac_parser"
>>   flv_muxer_select="aac_adtstoasc_bsf"
>>   gxf_muxer_select="pcm_rechunk_bsf"
>> +flif_demuxer_select="zlib exif"
>>   hds_muxer_select="flv_muxer"
> 
> This needs to be in the same commit as the one which adds the demuxer,
> it's a necessity for its build.
> 
> Moritz

These are actually optional dependencies for the demuxer, and these are
only required for metadata decoding. If not enabled, metadata is
skipped. Should I actually remove this and let the user voluntarily add
in the dependencies?

I now notice that removing the line generates some unused variable
warnings in the demuxer. I will take care of them in the next patch.

Will also do the suggested in the rest of the 2 Emails.

-- 
Thanks,
Anamitra



More information about the ffmpeg-devel mailing list