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

[Michael Niedermayer]

Hi

On Tuesday 03 May 2005 14:30, Rich Felker wrote:
> 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?

because all the other CONFIG_* are enable style too
the real problem IMHO is that many projects have their own configure script 
and duplicate all the stuff from the ffmpeg one, the current mess is just a 
small part of it, just look into the mplayer configure script, there is alot 
of stuff in it for lavc which is duplicated from ffmpeg/configure

maybe there should be a lavc-config which would help each projects configure 
script by turning the configure command line parameters into what needs to be 
added to config.h/mak?

[...]
-- 
Michael

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