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

[Aurelien Jacobs]

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.

Aurel