[MPlayer-dev-eng] [PATCH] merge all FFmpeg configure checks

Diego Biurrun diego at biurrun.de
Mon Sep 13 19:19:45 CEST 2010


On Sun, Sep 12, 2010 at 06:18:37PM +0200, Reimar Döffinger wrote:
> On Sun, Sep 12, 2010 at 05:59:18PM +0200, Diego Biurrun wrote:
> > I'd like to commit $attached unless I have overlooked something
> > important.  It merges all the separate FFmpeg library checks into
> > one.  I'm assuming that checking for libavcore assures us that the
> > FFmpeg version is new enough.  The diffstat is quite convincing:
> 
> Well, libavutil is a bit of an issue there.
> Let us ignore the included libavutil for a moment since 'we' (not
> me so much, but...) want to get rid of it at some point.
> Thing is, we need libavutil to compile no matter what.
> If there is only a single check for FFmpeg, that would then
> mean we always need FFmpeg.
> So I think either the libavutil check still needs to be
> separate or we could just as well get rid of the whole
> FFmpeg conditionals...

Here is a new patch that takes this into account.  Better?

Diego


More information about the MPlayer-dev-eng mailing list