By default, Bugzilla does not search the list of RESOLVED bugs.
You can force it to do so by putting the upper-case word ALL in front of your search query, e.g.: ALL tdelibs
We recommend searching for bugs this way, as you may discover that your bug has already been resolved and fixed in a later release.

Bug 141

Summary:	KDE xscreensaver integration missing files
Product:	TDE	Reporter:	Timothy Pearson <kb9vqf>
Component:	tdeartwork	Assignee:	Francois Andriot <albator78>
Status:	RESOLVED FIXED
Severity:	normal	CC:	albator78, bugwatch, darrella
Priority:	P3
Version:	3.5.11 [Trinity]
Hardware:	All
OS:	All
Compiler Version:		TDE Version String:
Application Version:		Application Name:
Attachments:	tdeartwork : fix xscreensaver directory

Description Timothy Pearson 2009-12-22 19:33:33 CST

KDE xscreensaver integration is missing .desktop files for some screensaver hacks, prohibiting their use under Trinity.

Comment 1 Timothy Pearson 2010-03-25 22:16:08 CDT

Fixed in SVN revision 1107577.

Comment 2 Francois Andriot 2013-07-31 04:27:45 CDT

Problem still exist in some distributions (Centos, Fedora ...)

Comment 3 Francois Andriot 2013-07-31 04:31:02 CDT

Created attachment 1431 [details]
tdeartwork : fix xscreensaver directory

The TDE build documentation says:
http://www.trinitydesktop.org/wiki/bin/view/Developers/HowToBuild

NOTE: Before building tdeartwork, install any and all xhack screensavers that might be uses, then
cd <tdeartwork>/kscreensaver/kxsconfig/
./update_hacks.sh

The problem is that the script "update_hacks.sh" has hardcoded path to "/usr/lib/xscreensaver", but most the distributions I build for use different directory.

The attached patch add the looking to different directory, it also add the removal of existing desktop-file which do not have the corresponding binary installed.

Comment 4 Darrell 2013-08-17 20:07:52 CDT

Francois, is this patch ready for R14 git?

Comment 5 Francois Andriot 2013-08-18 02:03:18 CDT

Yes, I currently use it for R14 on several distributions.

Comment 6 Darrell 2013-08-18 07:57:29 CDT

Patch pushed to git in commit 92b8071d.

Thanks for the patch!