FS#33944 - [mplayer] opening media files in gnome using mplayer.desktop floods ~/.cache/gdm/session.log

Attached to Project: Arch Linux
Opened by John (skeov) - Wednesday, 20 February 2013, 02:22 GMT
Last edited by Bartłomiej Piotrowski (Barthalion) - Thursday, 31 October 2013, 21:36 GMT
Task Type Bug Report
Category Packages: Extra
Status Closed
Assigned To Bartłomiej Piotrowski (Barthalion)
Architecture All
Severity Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

Description: I noticed that if I play files in mplayer using firefox "open with" download option, then ~/.cache/gdm/session.log can grow very large because of the output of mplayer. This though doesn't happen if I open a file from nautilus (I am using gnome). I'm guessing other DEs might have a similar issue.

I can see an older version of the package used to pass the -really-quiet option to mplayer so it generates no output but then it got removed.
https://projects.archlinux.org/svntogit/packages.git/commit/trunk?h=packages/mplayer&id=d73ff6beb888f8b2f45e19cd2a0deb9b0b8ca60e
https://projects.archlinux.org/svntogit/packages.git/commit/trunk?h=packages/mplayer&id=1d3c22b53e829dca79ecc560d625bacef2c67f1a

I think a better solution is to change Terminal=false to Terminal=true in mplayer.desktop so a terminal window opens up every time mplayer is run from its .desktop file so the output is only shown in it. This also makes more sense because otherwise if you play an audio-only file there is no obvious way to stop mplayer (and after all mplayer is a cli application).

Steps to reproduce:
1) wc -l ~/.cache/gdm/session.log
2) open a media file using firefox "open with" and choose MPlayer Media Player
3) let it play for some time (note that mplayer has constant output as long as a file plays)
4) wc -l ~/.cache/gdm/session.log
This task depends upon

Closed by  Bartłomiej Piotrowski (Barthalion)
Thursday, 31 October 2013, 21:36 GMT
Reason for closing:  No response
Comment by Ionut Biru (wonder) - Wednesday, 20 February 2013, 09:27 GMT
please report this upstream
Comment by John (skeov) - Wednesday, 20 February 2013, 19:54 GMT
This is not an upstream issue. arch provides the mplayer.desktop file.
Comment by Ionut Biru (wonder) - Wednesday, 20 February 2013, 21:41 GMT
it shouldn't be verbose as you said. maybe there is an issue with our latest mplayer and is a regression.
please provide logs.
Comment by John (skeov) - Wednesday, 20 February 2013, 23:01 GMT
There is nothing wrong with mplayer. The problem is the .desktop file of the package. Check the commits I linked. A better solution than passing the -really-quiet would be to use Terminal=true
   mplayer.desktop.diff (1.1 KiB)
Comment by Ionut Biru (wonder) - Wednesday, 20 February 2013, 23:22 GMT
for sure that option created some issues, that's why was removed.

please paste the output that is flooding your log.

Loading...