Bug 1019453 (sddm-kcm) - Review Request: sddm-kcm - SDDM KDE configuration module
Summary: Review Request: sddm-kcm - SDDM KDE configuration module
Keywords:
Status: CLOSED ERRATA
Alias: sddm-kcm
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Rex Dieter
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: kde-reviews F20Target-kde
TreeView+ depends on / blocked
 
Reported: 2013-10-15 18:22 UTC by Martin Bříza
Modified: 2013-11-19 21:45 UTC (History)
4 users (show)

Fixed In Version: sddm-kcm-0-0.1.20131114gitafdda33c.fc20
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2013-11-19 21:45:24 UTC
Type: ---
Embargoed:
rdieter: fedora-review+
gwync: fedora-cvs+


Attachments (Terms of Use)

Description Martin Bříza 2013-10-15 18:22:48 UTC
Spec URL: http://mbriza.fedorapeople.org/sddm/sddm-kcm.spec
SRPM URL: http://mbriza.fedorapeople.org/sddm/sddm-kcm-0.0.0-0.1.20131015gitafdda33c.fc20.src.rpm
Description: Graphical interface to configure SDDM using the KDE system settings.
Fedora Account System Username: mbriza

Comment 1 Rex Dieter 2013-10-15 18:36:26 UTC
Initial comments:

1. replace
%{_kde4_datadir}/kde4/apps/sddm-kcm/main.qml
with
%{_kde4_appsdir}/sddm-kcm/

2.  drop %config here, those really aren't user-editable config files
%config(noreplace)   %{_sysconfdir}/dbus-1/system.d/org.kde.kcontrol.kcmsddm.conf

Comment 2 Martin Bříza 2013-10-17 09:34:05 UTC
Updated:
Spec URL: http://mbriza.fedorapeople.org/sddm/sddm-kcm.spec
SRPM URL: http://mbriza.fedorapeople.org/sddm/sddm-kcm-0.0.0-0.1.20131015gitafdda33c.fc19.src.rpm

Notes:
1. There is no official release, just git. I'm not sure if this versioning will be fine - I just suppose sddm-kcm will use the same versioning scheme as sddm does.
2. It doesn't list themes but other settings work.

Comment 3 Kevin Kofler 2013-10-22 22:16:15 UTC
According to the guidelines, Version for completely unreleased packages should be 0, not 0.0.0.

Comment 4 Rex Dieter 2013-11-12 21:55:29 UTC
License: ok

1. Naming: mostly ok, please do reset to Version: 0 per comment #2

macros: ok

2. scriptlets: mostly ok, there's an extranous

%post -p /sbin/ldconfig
%postun -p /sbin/ldconfig

which is not needed, please remove.


The reset is clean and simple, APPROVED.


please address items 1,2 prior to doing any official builds.

Comment 5 Martin Bříza 2013-11-14 11:33:29 UTC
New Package SCM Request
=======================
Package Name: sddm-kcm
Short Description: SDDM KDE configuration module
Owners: mbriza than rdieter kkofler ltinkl jgrulich dvratil
Branches: f18 f19 f20
InitialCC:

Comment 6 Gwyn Ciesla 2013-11-14 12:58:12 UTC
Git done (by process-git-requests).

Comment 7 Fedora Update System 2013-11-14 16:16:31 UTC
sddm-kcm-0-0.1.20131114gitafdda33c.fc20 has been submitted as an update for Fedora 20.
https://admin.fedoraproject.org/updates/sddm-kcm-0-0.1.20131114gitafdda33c.fc20

Comment 8 Fedora Update System 2013-11-14 19:16:15 UTC
sddm-kcm-0-0.1.20131114gitafdda33c.fc20 has been pushed to the Fedora 20 testing repository.

Comment 9 Fedora Update System 2013-11-19 21:45:24 UTC
sddm-kcm-0-0.1.20131114gitafdda33c.fc20 has been pushed to the Fedora 20 stable repository.


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