Bug 846145 - Please adapt kdm for display manager rework feature
Please adapt kdm for display manager rework feature
Product: Fedora
Classification: Fedora
Component: kde-settings (Show other bugs)
Unspecified Unspecified
unspecified Severity unspecified
: ---
: ---
Assigned To: Rex Dieter
Fedora Extras Quality Assurance
Depends On:
Blocks: F18DisplayManager
  Show dependency treegraph
Reported: 2012-08-06 19:56 EDT by Lennart Poettering
Modified: 2012-08-08 11:37 EDT (History)
8 users (show)

See Also:
Fixed In Version: kde-settings-4.9-1
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Last Closed: 2012-08-08 09:45:46 EDT
Type: Bug
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---

Attachments (Terms of Use)
Proposed patch (2.73 KB, patch)
2012-08-08 08:25 EDT, Martin Bříza
no flags Details | Diff

  None (edit)
Description Lennart Poettering 2012-08-06 19:56:40 EDT

please update the kdm package to follow the new logic explained in the DisplayManagerRework page:


This requires shipping a systemd unit file in the kdm package as well as changes to the .spec file.

Please ping us if you need help, or have questions.

Note that this is an accepted F18 feature, we'd be thankful if you could complete this update in time for the F18 release.
Comment 1 Martin Bříza 2012-08-08 08:25:12 EDT
Created attachment 603009 [details]
Proposed patch

I'm attaching the proposed patch to fix this (Items 1, 2 and 9 from https://fedoraproject.org/wiki/Features/DisplayManagerRework). 
It works on my system as expected - KDE handles plymouth and it doesn't take a name on the org.freedesktop bus. Yet I'm not exactly sure if the .spec file changes are completely all right, please take a look at it and let me know if I did anything wrong.
Thank you.
Comment 2 Rex Dieter 2012-08-08 09:10:21 EDT
thanks, looks good.   I'll merge your work.
Comment 3 Rex Dieter 2012-08-08 09:45:46 EDT
Added to kde-settings-4.9-1
Comment 4 Lennart Poettering 2012-08-08 11:37:09 EDT
Patch looks good to me, too! Thanks for getting this done so quickly!

Note You need to log in before you can comment on or make changes to this bug.