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

[Diego Biurrun]

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.

Diego