Bug 234336

Summary:	XResetScreenSaver doesn't reset screensaver timer
Product:	[Unmaintained] kscreensaver	Reporter:	Clara Gnos <clara.gnos>
Component:	general	Assignee:	kscreensaver bugs tracking <kscreensaver-bugs-null>
Status:	RESOLVED FIXED
Severity:	normal
Priority:	NOR
Version:	unspecified
Target Milestone:	---
Platform:	Debian testing
OS:	Linux
Latest Commit:		Version Fixed In:
Sentry Crash Report:
Attachments:	Patch I used here to fix the problem

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