FS#5371 - mplayer gnome-screensaver support

Attached to Project: Arch Linux
Opened by Khashayar Naderehvandi (nightfrost) - Friday, 08 September 2006, 12:35 GMT
Last edited by Greg (dolby) - Saturday, 10 May 2008, 14:01 GMT
Task Type Feature Request
Category Packages: Current
Status Closed
Assigned To Thomas Bächler (brain0)
Architecture not specified
Severity Low
Priority Normal
Reported Version 0.7.2 Gimmick
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 2
Private No

Details

Please add the attached patch to mplayer in order to add support for gnome-screensaver. Without this patch the screensaver will start while mplayer is running.

Thanks.
   MPlayer-1.0pre8-gnome-screens... (9 KiB)
This task depends upon

Closed by  Greg (dolby)
Saturday, 10 May 2008, 14:01 GMT
Reason for closing:  Won't implement
Comment by Jan de Groot (JGC) - Friday, 08 September 2006, 12:58 GMT
The patch adds a dependency to dbus, which is in extra. We want to move dbus and dbus-glib to current when we move to gnome 2.16.
Comment by Khashayar Naderehvandi (nightfrost) - Friday, 08 September 2006, 13:07 GMT
I see. That's no good, as it feels mplayer will be semi-crippled under gnome. What's the best solution? Should I go ahead and add mplayer-gnome to the AUR and hope for it to enter community at some point? Or is there a better idea?
Comment by Jan de Groot (JGC) - Friday, 08 September 2006, 13:29 GMT
It will get implemented as soon as we move dbus and dbus-glib to current. dbus is a core component of nowadays desktop systems (gnome uses it completely from 2.16, KDE 4.x will also use it in the near future and more XFCE apps are switching every release), so having it in current isn't a bad idea IMHO.
Comment by Khashayar Naderehvandi (nightfrost) - Friday, 08 September 2006, 13:37 GMT
Oh, alright, you mean like that. I get it now. Great, I won't add anything to the AUR then. Thanks.
Comment by arjan timmerman (blaasvis) - Saturday, 11 November 2006, 11:45 GMT
please add it thomas !
Comment by Diego (wishmechaos) - Tuesday, 22 April 2008, 03:24 GMT
  • Field changed: Percent Complete (100% → 0%)
With current Mplayer (1.0pre2), gnome-screensaver isn't stopped. There's a patch in ABS, but it's for pre1 and doesn't work anymore.
Comment by Thomas Bächler (brain0) - Tuesday, 22 April 2008, 06:53 GMT
Last time I added the patch, it caused weird bugs for other people using gmplayer (not using gnome-screensaver). I am clearly against adding this again.
Comment by Diego (wishmechaos) - Tuesday, 22 April 2008, 20:19 GMT
I see. In any case, the problem is the quality of the patch. Ubuntu seems to have it working, but I don't know if it's the same patch. If it's clean, it could be ported from http://bazaar.launchpad.net/~ubuntu-dev/mplayer/ubuntu/changes . Else, feel free to close this bug.
Comment by Michał Kwidziński (kwidzin) - Thursday, 24 April 2008, 08:04 GMT
In attachment working patch with version: mplayer 1.0rc2-3
   MPlayer-gnome-screensaver.pat... (1.4 KiB)
Comment by Greg (dolby) - Saturday, 10 May 2008, 14:01 GMT
Please sent the patch upstream. Various similar patches have been rejected from the MPlayer developers in the past. As an alternative i propose you consider using gnome mplayer (http://dekorte.homeip.net/download/gnome-mplayer/ http://aur.archlinux.org/packages.php?ID=9498 http://gnome-mplayer.googlecode.com/)which has the feature built-in.