Opened 8 years ago

Last modified 8 years ago

#1890 new defect

useless error message about broken config file - should mention filename

Reported by: mplayer-bugs@… Owned by: reimar
Priority: normal Component: core
Version: 1.0rc4 Severity: normal
Keywords: Cc: compn
Blocked By: Blocking:
Reproduced by developer: Analyzed by developer:

Description

setup-pulseaudio --disable on openSUSE 11.4 breaks the /etc/mplayer/mplayer.conf file - see https://bugzilla.novell.com/show_bug.cgi?id=681113 for details.
That's something you can't do much about (it's a bug in setup-pulseaudio), I just mention it to give you some background information.

The problem is that mplayer displays a quite useless error message after disabling pulseaudio:

# mplayer
Option ,alsa needs a parameter at line 77

It took me quite some time to find out about which file the error message is.
Please add the filename to the error message. It should look like this:

# mplayer ### with enhanced error message
Option ,alsa needs a parameter at line 77 in /etc/mplayer/mplayer.conf

Please change the error message to include the filename.

Attachments (1)

patchconfig (2.7 KB) - added by cehoyos 8 years ago.
Patch to display config file name in config file error message

Download all attachments as: .zip

Change History (3)

Changed 8 years ago by cehoyos

Patch to display config file name in config file error message

comment:1 Changed 8 years ago by cehoyos

comment:2 Changed 8 years ago by compn

  • Cc patriotact@… added

carl, are you going to apply this? :)

Note: See TracTickets for help on using tickets.