← Back to team overview

registry team mailing list archive

[Bug 428884] Re: gnome-screensaver-command --poke no longer inhibits screensaver

 

Launchpad has imported 4 comments from the remote bug at
http://bugs.freedesktop.org/show_bug.cgi?id=25855.

If you reply to an imported comment from within Launchpad, your comment
will be sent to the remote bug automatically. Read more about
Launchpad's inter-bugtracker facilities at
https://help.launchpad.net/InterBugTracking.

------------------------------------------------------------------------
On 2010-01-02T02:34:15+00:00 L-lunak wrote:

Created an attachment (id=32398)
xserver patch

After xserver commit a32d717ac64b21e15a5736ae4981cdd32f2da56b
applications using XResetScreenSaver() no longer prevent the KDE
screensaver from activating, which uses XScreenSaverQueryInfo() to
detect the idle time and activates after a specified timeout. After this
commit there appears to be no way to detect that XResetScreenSaver() has
been used (the IDLE timer is presumably affected as well).

Testcase: The two attached test apps.
$ g++ -I/usr/include/X11R6 a.cpp -lX11 -o a.out 
$ g++ -I/usr/include/X11R6 b.cpp -lX11 -lXss -o b.out
$ ( sleep 20; ./a.out ) &
$ ./b.out
The time is not reset after 20 seconds even though it should.

Assuming a32d717ac64b21e15a5736ae4981cdd32f2da56b is correct, then I
suggest the attached patch.

Reply at: https://bugs.launchpad.net/gnome-
screensaver/+bug/428884/comments/97

------------------------------------------------------------------------
On 2010-01-02T02:34:36+00:00 L-lunak wrote:

Created an attachment (id=32399)
test app 1

Reply at: https://bugs.launchpad.net/gnome-
screensaver/+bug/428884/comments/98

------------------------------------------------------------------------
On 2010-01-02T02:34:53+00:00 L-lunak wrote:

Created an attachment (id=32400)
test app 2

Reply at: https://bugs.launchpad.net/gnome-
screensaver/+bug/428884/comments/99

------------------------------------------------------------------------
On 2010-02-12T15:02:13+00:00 Julien Cristau wrote:

Fixed in master, thanks for the report!

commit 001ce71dc11287dc94cc2fbc5d35677c046e6c04
Author: Julien Cristau <jcristau@xxxxxxxxxx>
Date:   Fri Feb 12 23:34:57 2010 +0100

    dix: restore lastDeviceEventTime update in dixSaveScreens
    
    This was removed in 6b5978dcf1f7ac3ecc2f22df06f7000f360e2066 (Do not
    reset lastDeviceEventTime when we do dixSaveScreens), but caused a
    regression for XResetScreenSaver.  Add the lastDeviceEventTime update
    back, but restrict it to that case.
    
    X.Org bug#25855 <http://bugs.freedesktop.org/25855>
    
    Reported-by: Lubos Lunak <l.lunak@xxxxxxx>
    Tested-by: Lubos Lunak <l.lunak@xxxxxxx>
    Signed-off-by: Julien Cristau <jcristau@xxxxxxxxxx>
    Signed-off-by: Keith Packard <keithp@xxxxxxxxxx>

Reply at: https://bugs.launchpad.net/gnome-
screensaver/+bug/428884/comments/127


** Changed in: xorg-server
   Importance: Unknown => Medium

-- 
gnome-screensaver-command --poke no longer inhibits screensaver
https://bugs.launchpad.net/bugs/428884
You received this bug notification because you are a member of Registry
Administrators, which is the registrant for Debian.