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 - KDE xscreensaver integration missing files
Summary: KDE xscreensaver integration missing files
Status: RESOLVED FIXED
Alias: None
Product: TDE
Classification: Unclassified
Component: tdeartwork (show other bugs)
Version: 3.5.11 [Trinity]
Hardware: All All
: P3 normal
Assignee: Francois Andriot
URL:
Depends on:
Blocks:
 
Reported: 2009-12-22 19:33 CST by Timothy Pearson
Modified: 2013-08-18 07:57 CDT (History)
3 users (show)

See Also:
Compiler Version:
TDE Version String:
Application Version:
Application Name:

Attachments
tdeartwork : fix xscreensaver directory (2.02 KB, patch)
2013-07-31 04:31 CDT, Francois Andriot 		Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
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!