[MPlayer-dev-eng] [PATCH] --enable-system-ffmpeg, was: linking external ffmpeg libs

[Diego Biurrun]

On Fri, Aug 13, 2010 at 06:14:02PM +0200, Reinhard Tartler wrote:
> On Fri, Aug 13, 2010 at 12:40:42 (CEST), Diego Biurrun wrote:
> 
> > On Thu, Aug 12, 2010 at 08:52:47AM -0400, Reinhard Tartler wrote:
> >> On Thu, Aug 12, 2010 at 07:59:40 (EDT), Diego Biurrun wrote:
> >> 
> >> > On Wed, Aug 11, 2010 at 09:34:39PM +0100, Edd Barrett wrote:
> >> >> 
> >> >> From a packager POV, linking mplayer to an existing ffmpeg package's
> >> >> libav* is an attractive option. Is there an easy way to not build
> >> >> mplayers internal libav*, but to use one in ffmpeg's shared objects?
> >> >> 
> >> >> If not, could I encourage an option in the build system?
> >> >
> >> > ./configure --help
> >> 
> >> I reckon he is asking for something like
> >> 
> >> $ ./configure --with-system-ffmpeg
> >> 
> >> as shortcut for the various --disable_libavfoo_a options. Diego, at
> >> FOSDEM, you've indicated sympathy with such a switch, has your opinion
> >> changed since then? If not, I'll try to come up with a patch for that
> >> next week when I'm back home.
> >
> > Go right ahead.
> 
> I don't know if this is the nicest patch, but it does work for me, and
> produces a much nicer configuration string:

I'm thinking that we should maybe just check for FFmpeg once and not for
each library separately.  What do you think?

> --- configure	(revision 31956)
> +++ configure	(working copy)
> @@ -355,6 +355,7 @@
>    --disable-libswscale_so   disable shared libswscale [autodetect]
> +  --enable-system-ffmpeg    disable shared ffmpeg libraries [no]

This looks reversed.

Diego