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

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


On Mon, Sep 13, 2010 at 07:19:45PM +0200, Diego Biurrun wrote:
> 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?

*sigh*

Diego
-------------- next part --------------
A non-text attachment was scrubbed...
Name: ffmpeg_check.diff
Type: text/x-diff
Size: 41905 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20100913/3c3f3657/attachment-0001.diff>


More information about the MPlayer-dev-eng mailing list