Bug 98819 - xscreensaver 4.22-r1 or 4.22-r2 not compatible with KDE screensaver support
Bug#: 98819 Product:  Gentoo Linux Version: unspecified Platform: x86
OS/Version: Linux Status: RESOLVED Severity: normal Priority: P2
Resolution: FIXED Assigned To: kde@gentoo.org Reported By: james.roberts-thomson@nbnz.co.nz
Component: Applications
URL: 
Summary: xscreensaver 4.22-r1 or 4.22-r2 not compatible with KDE screensaver support
Keywords:  
Status Whiteboard: 
Opened: 2005-07-12 14:40 0000
Description:   Opened: 2005-07-12 14:40 0000
I am using split ebuilds for my KDE build.

The kdeartwork-kscreensaver module cannot locate xscreensaver modules for
xscreensaver 4.22-r1 or 4.22-r2, and thus when compiled does not include any
xscreensavers.  RSS-Glx screensavers are included, but fail to work.

Also, if kdeartwork-kscreensaver was built agains an earlier xscreensaver
version, then upgrading to xscreensaver 4.22-r1 or later causes the xscreensaver
modules to fail.  (Well, actually I noticed my rss-glx screensaver break).

This issue seems to be caused by the xscreensaver 4.22-r1 compiling with
"--with-hackdir=/usr/lib/misc/xscreensaver".  There is a comment in the
ChangeLog along the lines of "changed hackdir to something more suitable". 
Unfortunately in this case, "more suitable" also meant "incompatible with KDE as
it stands".

Reproducible: Always
Steps to Reproduce:
1.  install or upgrade xscreensaver to v4.22-r1 or -r2
2.  re-emerge kdeartwork-kscreensaver
3.  notice that no xscreensaver modules appear in the KDE screensavers dialog

Actual Results:  
KDE "./configure" was unable to locate the xscreensaver modules, and dropped
xscreensaver support.  The CONTENTS file for kdeartwork-kscreensaver clearly
shows that no ".desktop" files for xscreensavers get installed.

Expected Results:  
"It should just work".

Seriously, either the xscreensaver hackdir change should be reverted, or the
kdeartwork-kscreensaver "configure" system (well, the automake/autoconf bits
presumably) need patching to be compatible with the new location.  I cannot
provide such a patch at this time, due to lack of familiarity with the auto*
toolset.

------- Comment #1 From James Roberts-Thomson 2005-07-12 14:43:20 0000 -------
Oh, I did discover a workaround:

cd /usr/lib; ln -s misc/xscreensaver xscreensaver

(but this kind-of negates the whole point of changing the hackdir!)

------- Comment #2 From Jakub Moc (RETIRED) 2005-07-12 15:41:47 0000 -------

*** This bug has been marked as a duplicate of 96074 ***

------- Comment #3 From Gregorio Guidi (RETIRED) 2005-07-12 16:00:17 0000 -------
Actually this is valid, the kdeartwork configure scripts need a fix. 

------- Comment #4 From Jure Repinc 2005-07-16 09:58:37 0000 -------
Bug about this on bugs.kde.org:
http://bugs.kde.org/show_bug.cgi?id=105748

------- Comment #5 From Kai Blaschke 2005-07-17 09:22:09 0000 -------
Created an attachment (id=63624) [details]
Patch resolving the xscreensaver install path issue

This patch resolves the problem of finding xscreensaver by adding an additional
check for lib/misc/xscreensaver/flame to the configure script. It should be
added to the official portage tree as soon as possible.

------- Comment #6 From Peter Hyman 2005-07-27 00:57:09 0000 -------
x11-misc/fireflies also has /usr/lib/xscreensaver hard coded in its ebuild,
which could cause a problem unless /usr/lib/xscreensaver is linked.

In addition, during configure, this error occurs. I think it needs to be
/usr/lib/misc/xscreensaver, not /usr/lib/xscreensaver.

ln: creating symbolic link
'/var/tmp/portage/xscreensaver-4.22-r3/image//usr/lib/xscreensaver/config' to
/usr/share/xscreensaver/config': No such file or directory

HTH

------- Comment #7 From Dan Armak (RETIRED) 2005-07-29 11:18:31 0000 -------
> In addition, during configure, this error occurs. 
You mean in fireflies? Please file a separate bug for that and assign to the 
maintainer. 

------- Comment #8 From Dan Armak (RETIRED) 2005-07-29 11:22:26 0000 -------
Gregorio, I noticed you submitted a patch at the kde.org bug that adds a 
configure switch to specify the hacks dir - why not commit it to portage? 

------- Comment #9 From Gregorio Guidi (RETIRED) 2005-07-30 01:33:11 0000 -------
(In reply to comment #8) 
> Gregorio, I noticed you submitted a patch at the kde.org bug that adds a  
> configure switch to specify the hacks dir - why not commit it to portage?  
 
I committed it to kdeartwork, and I was just committing it to 
kdeartwork-kscreensaver as we speak... So this is fixed now. 

------- Comment #10 From Gregorio Guidi (RETIRED) 2005-08-04 02:05:07 0000 -------
*** Bug 101261 has been marked as a duplicate of this bug. ***

------- Comment #11 From Gregorio Guidi (RETIRED) 2005-08-12 01:58:48 0000 -------
*** Bug 102177 has been marked as a duplicate of this bug. ***

------- Comment #12 From Matt T. Proud 2005-08-28 01:56:00 0000 -------
(In reply to comment #9)
> (In reply to comment #8) 
> > Gregorio, I noticed you submitted a patch at the kde.org bug that adds a  
> > configure switch to specify the hacks dir - why not commit it to portage?  
>  
> I committed it to kdeartwork, and I was just committing it to 
> kdeartwork-kscreensaver as we speak... So this is fixed now. 

Are you entirely sure that this is fixed, because xscreensaver-4.22-r2 still
does not register with kdeartwork-kscreensaver-3.4.1.

Here is a snippet from a build of kdeartwork-kscreensaver-3.4.1. from a few
minutes ago:

checking for lib64/xscreensaver/flame... not found
checking for lib/xscreensaver/flame... not found
checking for bin/xscreensaver-hacks/flame... not found
checking for hacks/flame... not found
checking if kscreensaver should be compiled... yes
configure: creating ./config.status
fast creating Makefile
fast creating kscreensaver/Makefile
fast creating kscreensaver/kdesavers/Makefile
fast creating kscreensaver/kdesavers/data/Makefile
fast creating kscreensaver/kpartsaver/Makefile
fast creating kscreensaver/kxsconfig/Makefile
fast creating kscreensaver/kxsconfig/ScreenSavers/Makefile
fast creating kscreensaver/xsavers/Makefile
config.pl: fast created 8 file(s).
config.status: creating config.h
config.status: executing depfiles commands

xscreensaver could not be detected on your system.
Check www.jwz.org/xscreensaver if you're loving to
have tons of screensavers in KDE

I would be inclined to reopen the bug for investigation.

------- Comment #13 From Jason Dolan 2005-08-29 06:36:04 0000 -------
> I would be inclined to reopen the bug for investigation.
>

I agree, an emerge of xscreensaver-4.22-r2 and kdeartwork-kscreensaver-3.4.1
(both with +kde +xscreensaver +opengl USE flags) did NOT result in the
xscreensavers being accessable with kscreensaver.

The suggested work around:

cd /usr/lib; ln -s misc/xscreensaver xscreensaver

then re-emerge kscreensaver worked however.  So it seems the configure script
still does not look in /usr/lib/misc/xscreensaver.