Bug 234336 - XResetScreenSaver doesn't reset screensaver timer
Summary: XResetScreenSaver doesn't reset screensaver timer
Status: RESOLVED FIXED
Alias: None
Product: kscreensaver
Classification: Miscellaneous
Component: general (show other bugs)
Version: unspecified
Platform: Debian testing Linux
: NOR normal
Target Milestone: ---
Assignee: kscreensaver bugs tracking
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-04-14 13:36 UTC by Clara Gnos
Modified: 2010-04-17 10:50 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:


Attachments
Patch I used here to fix the problem (641 bytes, patch)
2010-04-14 19:56 UTC, Clara Gnos
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Clara Gnos 2010-04-14 13:36:57 UTC
Version:           4.4.2 (using KDE 4.4.2)
OS:                Linux
Installed from:    Debian testing/unstable Packages

I noticed that there was a bug at https://bugs.freedesktop.org/show_bug.cgi?id=25855 that is fixed here (xorg-xserver 1.7.6 has the patch cherry picked) and still the kscreensaver starts. I had the two example applications attached to the xorg application running (a started using `while true; do sleep 20; ./a.out done`) and kscreensaver still starts. 

I see that the idle timer is decreased every ~20 seconds but after a minute kscreensaver still starts (saw 2-3 idle timer decreases in that time).

So it is more or less the bug report already at #214977, but I can show that this is not the fault of mplayer (not used here) or xserver.
Comment 1 Clara Gnos 2010-04-14 13:49:24 UTC
There is a ; missing after "./a.out". Please add it before using it as example.

sandsmark also told me in #kde-devel that this is probably a regression in kde 4.4 and that he noticed it too with mplayer
Comment 2 Clara Gnos 2010-04-14 16:53:34 UTC
Commit 1080356 looks interesting to me. http://websvn.kde.org/trunk/KDE/kdebase/workspace/CMakeLists.txt?r1=1073015&r2=1080356 this stuff is needed for krunner/config-xautolock.h.cmake

HAVE_XSCREENSAVER is still needed for ./krunner/screensaver/xautolock.cpp - so it is build without XSCREENSAVER support.... bad :(
Comment 3 Clara Gnos 2010-04-14 19:56:25 UTC
Created attachment 42788 [details]
Patch I used here to fix the problem

I rebuild the debian packages here with that patch to re-enable the HAVE_XSCREENSAVER stuff again in cmake.
Comment 4 Clara Gnos 2010-04-14 20:14:20 UTC
Review request was sent to http://reviewboard.kde.org/r/3605/
Comment 5 Oswald Buddenhagen 2010-04-17 10:50:39 UTC
SVN commit 1115717 by ossi:

re-add xscreensaver check

BUG: 234336

based on http://reviewboard.kde.org/r/3605



 M  +2 -0      CMakeLists.txt  


WebSVN link: http://websvn.kde.org/?view=rev&revision=1115717