456062 – kde-base/systemsettings-4.10.0 requires dependency on kwin

Bug 456062 - kde-base/systemsettings-4.10.0 requires dependency on kwin

Summary: kde-base/systemsettings-4.10.0 requires dependency on kwin

Status:	RESOLVED FIXED

Alias:	None

Product:	Gentoo Linux
Classification:	Unclassified
Component:	[OLD] KDE (show other bugs)
Hardware:	All Linux

Importance:	Normal normal
Assignee:	Gentoo KDE team

URL:
Whiteboard:
Keywords:

Depends on:
Blocks:

Reported:	2013-02-07 18:20 UTC by theodor
Modified:	2013-02-08 14:15 UTC (History)
CC List:	2 users (show)

See Also:
Package list:
Runtime testing required:	---

Attachments
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 theodor 2013-02-07 18:20:26 UTC

When I haven't installed kwin, emerging systemsettings fails with the linker error:
/usr/lib/gcc/i686-pc-linux-gnu/4.7.2/../../../../i686-pc-linux-gnu/bin/ld: cannot find -lkdecorations
collect2: error: ld returned 1 exit status
make[2]: *** [lib/kcm_style.so] Error 1

Installing kwin solves this. So please add a dependency to systemsettings for kwin.

Reproducible: Always

Comment 1 Franz Trischberger 2013-02-08 07:31:44 UTC

kwin really seems to be needed:

kcontrol/style/kcmstyle.cpp:442

        QString buttonsOnLeft = kwinConfig.readEntry("ButtonsOnLeft", KDecorationOptions::defaultTitleButtonsLeft());
        QString buttonsOnRight = kwinConfig.readEntry("ButtonsOnRight", KDecorationOptions::defaultTitleButtonsRight());

A patch to remove kwin-dep would remove kcm_style - which I think nobody wants ;)

Comment 2 Michael Palimaka (kensington) gentoo-dev

2013-02-08 14:15:19 UTC

Thanks, fixed in CVS.

+  08 Feb 2013; Michael Palimaka <kensington@gentoo.org>
+  systemsettings-4.10.0.ebuild:
+  Add missing dependencies wrt bug #456062.