[MPlayer-dev-eng] [PATCH] Do not read codecs.conf files by default

[Diego Biurrun]

On Wed, Jan 17, 2007 at 10:35:50AM +0100, Reimar Doeffinger wrote:
> On Wed, Jan 17, 2007 at 12:16:37AM +0100, Diego Biurrun wrote:
> > On Mon, Jan 15, 2007 at 11:01:34PM +0100, Reimar Döffinger wrote:
> > > A possible consideration would be to change codecs.conf in the sources
> > > so that it would not load without manual modification by the user,
> > > a I-know-what-I-am-doing similar to the lavf bframes thing...
> > 
> > That's precisely what my patch does :)  It forces the user to provide a
> > command line option (or config file entry) if a codecs.conf file is to
> > be loaded, otherwise it is not loaded...
> 
> No, it's not precisely the same, since 1) your patch requires modifying
> a _different_ file 2) as you say, it makes codecs.conf a non-config
> files. That is fine. But your patch does not tell the user about this
> fact! I am almost sure that next we would get bug reports about users
> editing codec.conf to get some new windows codec working but it doesn't
> work...

That's what release notes are for.  This is not the first time that
MPlayer changes behavior ...

Diego