[MPlayer-dev-eng] Error with MEncoder configuration file

[Ivan Kalvachev]

2006/3/13, Diego Biurrun <diego at biurrun.de>:
> On Sun, Mar 12, 2006 at 04:50:50PM +0100, Benjamin Zores wrote:
> >
> > According to the CVS Man page :
> >
> > "The configuration file for MEncoder is ???mencoder.conf???  in
> > your  configuration  directory  (e.g.  /etc/mplayer  or /usr/local/etc/
> > mplayer), the user specific one is ???~/.mplayer/mencoder."
> >
> > According to the mencoder.c source :
> >
> > void me_parse_cfgfiles( m_config_t* conf )
> > {
> >   char *conffile;
> >   if ((conffile = get_path("mencoder")) == NULL) {
> >     mp_msg(MSGT_CPLAYER,MSGL_ERR,MSGTR_GetpathProblem);
> >   } else {
> >     if (m_config_parse_config_file(conf, conffile) < 0)
> >       mencoder_exit(1,MSGTR_ConfigfileError);
> >     free(conffile);
> >   }
> > }
> >
> > As a result, /etc/mplayer/mencoder is recognized as well as ~/.mplayer/mencoder.
> > But /etc/mplayer/mencoder.conf isn't recognized as it should be.
> >
> > Mistake in code (i vote for) or typo in MAN page ?
>
> I vote for changing it to mencoder.conf.  I think we had weird problems
> with MEncoder trying to parse itself as configuration file on Windows or
> something.

Agree, maybe we should rename mplayer to mplayer.conf too.