Bug 846145 - Please adapt kdm for display manager rework feature
Summary: Please adapt kdm for display manager rework feature
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: kde-settings
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Rex Dieter
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: F18DisplayManager
TreeView+ depends on / blocked
 
Reported: 2012-08-06 23:56 UTC by Lennart Poettering
Modified: 2012-08-08 15:37 UTC (History)
8 users (show)

Fixed In Version: kde-settings-4.9-1
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2012-08-08 13:45:46 UTC
Type: Bug


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

Description Lennart Poettering 2012-08-06 23:56:40 UTC
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 12:25:12 UTC
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 13:10:21 UTC
thanks, looks good.   I'll merge your work.

Comment 3 Rex Dieter 2012-08-08 13:45:46 UTC
Added to kde-settings-4.9-1

Comment 4 Lennart Poettering 2012-08-08 15:37:09 UTC
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.