Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 165404 - media-video/mplayer - adding gnome-screensaver support to mplayer
Summary: media-video/mplayer - adding gnome-screensaver support to mplayer
Status: RESOLVED WONTFIX
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: High enhancement
Assignee: Gentoo Media-video project
URL: http://lists.mplayerhq.hu/pipermail/m...
Whiteboard:
Keywords:
: 176725 203615 207377 (view as bug list)
Depends on:
Blocks:
 
Reported: 2007-02-05 06:28 UTC by Ivan S. Dubrov
Modified: 2008-02-27 09:49 UTC (History)
4 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Ivan S. Dubrov 2007-02-05 06:28:06 UTC
Please consider adding gnome-screensaver support to mplayer. Here is the patch http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/2006-October/046721.html

Although upstream does not accept this patch I think including it into the ebuild is not a big issue and will be really beneficial for GNOME users. 

Reproducible: Always
Comment 1 Steve Dibb (RETIRED) gentoo-dev 2007-02-05 14:56:18 UTC
Generally speaking, I dont want to import patches into our code that adds new functionality, since then we'll be stuck maintaining that new featureset ourselves.

I would prod upstream again to see what they think of the idea (I think it's a good one), and get them to go over the issue again.

Failing that, I'd put this in an overlay instead.

Sorry, Ivan.
Comment 2 Reimar Döffinger 2007-02-05 15:06:19 UTC
I would very much prefer if instead the most basic options were added to screensavers, like "don't start if program XY is running" or "don't start if window X covers more that Y% of the screen" instead of having to include screensaver handling for at least 3 screensavers in all media players, presentation programs etc. pp.
IOW: I would like it if someone could force those who created this mess to deal with it.
Comment 3 Jakub Moc (RETIRED) gentoo-dev 2007-05-01 20:23:08 UTC
*** Bug 176725 has been marked as a duplicate of this bug. ***
Comment 4 Jakub Moc (RETIRED) gentoo-dev 2007-12-28 20:55:47 UTC
*** Bug 203615 has been marked as a duplicate of this bug. ***
Comment 5 Samuli Suominen (RETIRED) gentoo-dev 2008-01-25 12:48:40 UTC
*** Bug 207377 has been marked as a duplicate of this bug. ***
Comment 6 Reimar Döffinger 2008-01-26 19:43:56 UTC
MPlayer SVN supports and uses the XResetScreenSaver and XScreenSaverSuspend methods.
For those screensavers that stubbornly refuse to support these you can use the -heartbeat-cmd option to specify an arbitrary command that will be executed periodically while MPlayer runs.
No further ways (unless maybe if they have the blessing of the X.org crew) to support screensaver handling will be added.
Comment 7 Erik 2008-02-27 09:49:34 UTC
Thanks Gentoo for making a better decision than some other distributions! (If a distribution wants to fix a bug in gnome-screensaver, it should be fixed in gnome-screensaver, not in mplayer.)