Bug 846145

Summary: Please adapt kdm for display manager rework feature
Product: [Fedora] Fedora Reporter: Lennart Poettering <lpoetter>
Component: kde-settingsAssignee: Rex Dieter <rdieter>
Status: CLOSED RAWHIDE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: rawhideCC: extras-orphan, jreznik, kevin, ltinkl, rdieter, rnovacek, smparrish, than
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: kde-settings-4.9-1 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2012-08-08 09:45:46 EDT Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---
Bug Depends On:    
Bug Blocks: 846137    
Attachments:
Description Flags
Proposed patch none

Description Lennart Poettering 2012-08-06 19:56:40 EDT
Heya,

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

https://fedoraproject.org/wiki/Features/DisplayManagerRework

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

Hello, 
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!