[Buildroot] [PATCH] mplayer: needs zlib

[Gustavo Zacarias <gustavo@zacarias.com.ar>]

On 13/10/15 17:57, Thomas Petazzoni wrote:

> While this solution is simple, I am not sure if this is the right one.
> It is in fact perfectly possible to build mplayer without zlib, by
> doing:
>
> +MPLAYER_CONF_OPTS += --disable-decoder=apng --disable-encoder=apng --disable-decoder=tdsc
>
> So in fact, zlib is only needed for two decoders/encoders in the ffmpeg
> code. However, it is not really clear to me how to handle this
> properly: there is one configure script at the top-level directory, and
> then another one in the ffmpeg/ directory. And it is really the ffmpeg
> encoders/decoders that have this zlib requirement (and probably
> others?).
>
> So, simple solution is to make zlib mandatory (which is reasonable:
> zlib is small, and probably already part of most systems). However, I
> wonder if there are not some other dependencies for certain
> encoders/decoders.
>
> Bernd, Gustavo, do you have some suggestions ?

Hi Bernd, Thomas, Vicente.
Both apng and tdsc are on the same family of codecs, they're basically 
animated-image formats, apng being animated png, and tdsc being a form 
of animated jpeg.
Since in general we don't require any special deps for mplayer (and it 
doesn't require them most of the time) i'd say being able to build 
without zlib would be desirable, specially since these formats are 
unlikely to normally be used via mplayer (they probably weren't an 
available feature with 1.1.x)
So using an autodep on zlib seems better IMO.
Regards.