Bug 85307

Summary:	kdeartwork/screensaver/kdesavers/firesaver won't compile without arts
Product:	[Unmaintained] kscreensaver	Reporter:	Laurence Withers <l>
Component:	general	Assignee:	Caleb Tennis <caleb>
Status:	RESOLVED FIXED
Severity:	normal	CC:	benkstein, caleb
Priority:	NOR
Version First Reported In:	unspecified
Target Milestone:	---
Platform:	Compiled Sources
OS:	Linux
Latest Commit:		Version Fixed/Implemented In:
Sentry Crash Report:

Description Laurence Withers 2004-07-16 02:24:52 UTC

Version:            (using KDE KDE 3.2.91)
Installed from:    Compiled From Sources
Compiler:          gcc 3.3.3 Gentoo release 6
OS:                Linux

While compiling kdeartwork-3.3.0_beta1, having run the configure script with
--without-arts, the "firesaver" srceensaver won't compile, since it tries to
include some arts headers.

If the screensaver does actually require arts, then the configure script should
simply not compile that particular screen saver.

Comment 1 Clara Gnos 2004-07-26 21:24:36 UTC

Same problem with kdeartwork-3.3.0_beta2 (3.2.92). It was also reported here: http://bugs.gentoo.org/show_bug.cgi?id=58262

It should check with automake if the headers exists + (if they exists) --without-arts

Comment 2 Caleb Tennis 2004-08-13 15:26:41 UTC

This should now be fixed in HEAD - will backport to 3.3 after 3.3.0 is released

Comment 3 Laurence Withers 2004-08-13 19:32:10 UTC

I don't suppose that there is any chance of this getting fixed for the 3.3.0
release? If not, there will be several Gentoo users who get bitten by this,
because Portage (the Gentoo packaging system) now provides the option to not
use arts with KDE.

Thanks for your time so far.

Comment 4 Stephan Kulow 2004-08-13 19:36:20 UTC

Then it's good Portage doesn't provide a way to kill someone!

Comment 5 Caleb Tennis 2004-08-13 19:59:55 UTC

Whether the fix is included in 3.3.0 is coolo's decision - however, the fix may be incorporated into Gentoo - but that's a matter of discussion there.

http://bugs.gentoo.org/show_bug.cgi?id=58822

Comment 6 Caleb Tennis 2004-08-26 16:47:04 UTC

I plan to commit these changes to the 3.3 branch soon unless someone tells me otherwise.

Comment 7 Chris Howells 2004-08-27 22:02:38 UTC

Caleb fixed this in CVS HEAD; Caleb, can you also please backport to BRANCH