Bug 520236 – kde-base/kdm: split out greeters request

• Status: RESOLVED FIXED
• Product: Gentoo Linux

[Description Andreas Hermann 2014-08-19 09:15:14 UTC]

Created attachment 383106 [details]
New libkgreeterplugin ebuild

KDE in gentoo currently hard depends on the kdm display manager, because kde-base/ksmserver has RDEPEND=kde-base/kdm. This has been introduced with #496306, as the KDE screenlocker uses libs (greeter plugins) which are currently build by the kdm ebuild.
This is a packaging issue, as the greeters can be build separately from kdm. This is especially useful, as kde-base/kdebase-meta-4.13.3 (which is in stabilization right now) introduces the 'display-manager' use flag, so the user can choose a different DM (lightdm/sddm). But currently, kdm is built anyway with ksmserver, so when using lighdm for example, one has two kcm modules for DM in system settings (ugly and not needed).

Proposal:
1.) Split out the greeter plugins from kdm ebuild, call it libkgreeterplugin?
2.) Make ksmserver rdepend on the new libkgreeterplugin?

I have attached three ebuilds as proposal.

Thoughts?

[Comment 1 Andreas Hermann 2014-08-19 09:16:43 UTC]

Created attachment 383108 [details]
Updated ksmserver ebuild (rdepend on libkgreeterplugin)

[Comment 2 Andreas Hermann 2014-08-19 09:17:33 UTC]

Created attachment 383110 [details]
Updated kdm ebuild (don't build greeter plugins)

[Comment 3 Johannes Huber (RETIRED) 2014-08-19 10:25:28 UTC]

Comment on attachment 383108 [details]
Updated ksmserver ebuild (rdepend on libkgreeterplugin)

Please attach a patch.

[Comment 4 Johannes Huber (RETIRED) 2014-08-19 10:25:47 UTC]

Comment on attachment 383110 [details]
Updated kdm ebuild (don't build greeter plugins)

Please attach a patch.

[Comment 5 Andreas Hermann 2014-08-19 11:24:51 UTC]

(In reply to Johannes Huber from comment #3)
> Comment on attachment 383108 [details]
> Updated ksmserver ebuild (rdepend on libkgreeterplugin)
> 
> Please attach a patch.

Sorry, attached two proposed patches for kde-base/ksmserver and kde-base/kdm ebuilds.

[Comment 6 Andreas Hermann 2014-08-19 11:25:23 UTC]

Created attachment 383114 [details, diff]
patch for ksmserver

[Comment 7 Andreas Hermann 2014-08-19 11:25:39 UTC]

Created attachment 383116 [details, diff]
patch for kdm

[Comment 8 Andreas Hermann 2014-08-20 11:53:20 UTC]

I have tested this on my local overlay, no problems so far.

[Comment 9 Johannes Huber (RETIRED) 2014-08-25 08:15:11 UTC]

*** Bug 520676 has been marked as a duplicate of this bug. ***

[Comment 10 Philip P. 2014-09-23 13:42:15 UTC]

*** Bug 523340 has been marked as a duplicate of this bug. ***

[Comment 11 Johannes Huber (RETIRED) 2015-03-15 13:32:27 UTC]

Thank you for reporting and your intial work. Andreas Sturmlechner made a pull request via github, which is now in kde overlay.

[Comment 12 Andreas Hermann 2015-03-15 17:15:54 UTC]

(In reply to Johannes Huber from comment #11)
> Thank you for reporting and your intial work. Andreas Sturmlechner made a
> pull request via github, which is now in kde overlay.

You are welcome. Happy to see this in kde overlay. Thanks.

[Comment 13 Johannes Huber (RETIRED) 2015-03-29 11:40:25 UTC]

Thank you all. KDE Workspace 4.11.17 is now in cvs. Please sync in some hours to get the changes.