[FFmpeg-cvslog] r8949 - in trunk: configure libavcodec/Makefile libavcodec/wmv2.c

[Aurelien Jacobs]

On Thu, 10 May 2007 11:23:31 +0200
Aurelien Jacobs <aurel at gnuage.org> wrote:

> On Thu, 10 May 2007 02:07:37 +0200
> Michael Niedermayer <michaelni at gmx.at> wrote:
> 
> > Hi
> > 
> > On Thu, May 10, 2007 at 01:52:23AM +0200, Diego Biurrun wrote:
> > > On Thu, May 10, 2007 at 01:24:07AM +0200, Aurelien Jacobs wrote:
> > > > 
> > > > To be more precise, the following config is enough to show the bug:
> > > > ./configure --disable-decoder=msmpeg4v1 --disable-decoder=msmpeg4v2 \
> > > >             --disable-decoder=msmpeg4v3 --disable-encoder=msmpeg4v1 \
> > > >             --disable-encoder=msmpeg4v2 --disable-encoder=msmpeg4v3
> > > > 
> > > > IOW just disable compilation of msmpeg4.o and see all the other codecs
> > > > which depends on it failing to link.
> > > > To fix this you may need to make compilation of mpegvideo.c optional
> > > > (and this would require at the very least a long dependency list).
> > > 
> > > Yes, I've been investigating this myself today.  Maybe we should make
> > > the compilation of msmeg4.c unconditional again.
> 
> IIRC, this would require to make compilation of h263dec.c unconditional
> again :-(
> 
> > >  Untangling the
> > > dependencies is going to be tricky.  On the other hand it's probably
> > > well worth it in the long run...
> > 
> > iam in favor of correct dependancies and maybe even spliting files up
> > a little if that would help ...
> 
> I absolutely agree. It will probably be a bit tricky and it will require
> some time, but I think it's worth trying to do it.

Fixed :-)
(it wasn't that hard after all)

Aurel