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

[Rich Felker]

On Tue, May 03, 2005 at 02:43:16AM +0200, Michael Niedermayer wrote:
> 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

This is a huge burden for every calling program.. Why not instead get
rid of the CONFIG_* for encoders and instead make CONFIG_DISABLE_* so
that the default is enable?

Rich