[Ffmpeg-cvslog] CVS: ffmpeg configure,1.170,1.171

[Michael Niedermayer]

Hi

On Tuesday 03 May 2005 01:45, Rich Felker wrote:
> On Mon, May 02, 2005 at 10:48:25PM +0200, Michael Niedermayer CVS wrote:
> > Update of /cvsroot/ffmpeg/ffmpeg
> > In directory mail:/var2/tmp/cvs-serv3770
> >
> > Modified Files:
> >  configure
> > Log Message:
> > also put the CONFIG_FOOBAR_EN/DECODER stuff in config.mak
> >
> >
> > Index: configure
> > ===================================================================
> > RCS file: /cvsroot/ffmpeg/ffmpeg/configure,v
> > retrieving revision 1.170
> > retrieving revision 1.171
> > diff -u -d -r1.170 -r1.171
> > --- configure 2 May 2005 14:24:41 -0000 1.170
> > +++ configure 2 May 2005 20:48:23 -0000 1.171
> > @@ -1590,6 +1590,7 @@
> >
> >  for codec in $CODEC_LIST ; do
> >      echo "#define CONFIG_`echo $codec | awk '{$1=toupper($1)}1'` 1" >>
> > $TMPH +    echo "CONFIG_`echo $codec | awk '{$1=toupper($1)}1'`=yes" >>
> > config.mak done
>
> I have a feeling this has broken lavc usage in MPlayer.. :)
> IMO there should be some CONFIG_ALL_CODECS the calling build system
> can use..

IMHO no, maintaining the CONFIG_ALL_CODECS code would require an additional 
thing to be changed each time a new codec is added, mplayers configure should 
be changed so codecs can be disabled individually

[...]
-- 
Michael

"nothing is evil in the beginning. Even Sauron was not so." -- Elrond