[MPlayer-dev-eng] [PATCH] PNG_ENCODER needed for vf_screenshot with --disable-mencoder

[Magnus Damm]

On Mon, Aug 11, 2008 at 6:16 PM, Magnus Damm <magnus.damm at gmail.com> wrote:
> On Mon, Aug 11, 2008 at 3:38 AM, Diego Biurrun <diego at biurrun.de> wrote:
>> On Sun, Aug 10, 2008 at 03:22:49PM +0200, Adrian Stutz wrote:
>>> When compiling mplayer without mencoder, only mpeg1 and snow encoders are
>>> included.
>>> The png encoder is missing, which is needed for vf_screenshot.
>>
>> Applied.  Next time please attach patches, your mailer mangles them.
>
> This patch breaks building without zlib. Revision 27448 is broken,
> 27441 builds just fine for me.
>
> The header file check for zlib is done early, but PNG_ENCODER is set
> regardless the result of that check. Not sure how to fix that in a
> clean way.

Anyway, here's a patch to fix the build problem the ugly way.

/ magnus
-------------- next part --------------
A non-text attachment was scrubbed...
Name: mplayer-svn-27454-no-zlib-20080812.patch
Type: text/x-patch
Size: 513 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20080812/5f1db67a/attachment.bin>