[MPlayer-dev-eng] [PATCH] add VP8/WebM support

[Diego Biurrun]

On Tue, May 25, 2010 at 01:15:15PM -0400, James Zern wrote:
> On Wed, May 19, 2010 at 16:46, Reimar Döffinger
> <Reimar.Doeffinger at gmx.de> wrote:
> > I am not the maintainer but the mkv change looks ok to me.
> > The configure change should probably away review of the FFmpeg patch and
> > then be modeled based on what came out of that discussion.
> >
> Updated to match vpx decoder name and libvpx include file location.
> The demux_mkv changes are provided separately as well.

> --- configure	(revision 31217)
> +++ configure	(working copy)
> @@ -7596,6 +7600,39 @@ else
>  
> +    _inc_vpx=
> +    cc_check $_inc_vpx $_ld_vpx            &&
> +    extra_cflags="$extra_cflags $_inc_vpx" &&

This is nonsense, _inc_vpx is empty!

Other than that the patch looks good.

Diego