iaddis / electricsheep

Automatically exported from code.google.com/p/electricsheep
1 stars 1 forks source link

mplayer not being called with -nostop-xscreensaver, and so DPMS is being disabled #2

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Set a DPMS standby/suspend/off timeout on a machine with an X11 server
2. Allow electricsheep to run for at least that amount of time

What is the expected output? What do you see instead?

The monitors should standby/suspend/turn off. Instead, the monitors stay on.

What version of the product are you using? On what operating system?

electricsheep v2.7b12 on Linux machines (Fedora 15 and Arch Linux) with an X11 
server. This problem should affect all versions of electricsheep shelling out 
to mplayer to display the screen saver in X11.

Please provide any additional information below.

The:

-nostop-xscreensaver

option should be added to the command-line arguments for mplayer. Other screen 
savers that use mplayer have this behavior, and electricsheep should follow 
suit.

As of yesterday, mplayer was patched (SVN r34074) to start paying attention to 
this option again. At the time of the patch, a regression was introduced that 
made nostop-xscreensvaer the default (and so the option wasn't necessary). This 
regression will be fixed and the default will be stop-xscreensaver. So 
electricsheep needs to make the configuration option explicit.

Original issue reported on code.google.com by ted.pav...@gmail.com on 7 Sep 2011 at 3:33

GoogleCodeExporter commented 9 years ago
I see now that when electricsheep v2.7b12 is called as a screensaver, it does 
properly use the nostop-xscreensaver flag. So there is no bug.

This issue can be closed.

Original comment by ted.pav...@gmail.com on 7 Sep 2011 at 3:48

GoogleCodeExporter commented 9 years ago
as noted here http://electricsheep.org/node/2837
this was really a bug in mplayer, but it's been fixed.
but we aren't developing the old linux version anymore,
just the new cross-playform code.

Original comment by scottdra...@gmail.com on 29 Sep 2011 at 11:36