Bug 1094331 (kf5-kdbusaddons) - Review Request: kf5-kdbusaddons - KDE Frameworks 5 Tier 1 addon with various classes on top of QtDBus
Summary: Review Request: kf5-kdbusaddons - KDE Frameworks 5 Tier 1 addon with various ...
Keywords:
Status: CLOSED RAWHIDE
Alias: kf5-kdbusaddons
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Jan Grulich
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: kde-reviews kf5
TreeView+ depends on / blocked
 
Reported: 2014-05-05 13:14 UTC by Daniel Vrátil
Modified: 2015-11-02 01:37 UTC (History)
4 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2014-05-19 09:06:00 UTC
Type: ---
Embargoed:
jgrulich: fedora-review+
gwync: fedora-cvs+


Attachments (Terms of Use)

Description Daniel Vrátil 2014-05-05 13:14:16 UTC
Spec URL: http://dvratil.fedorapeople.org/kf5/review/kf5-kdbusaddons.spec
SRPM URL: http://dvratil.fedorapeople.org/kf5/review/kf5-kdbusaddons-4.99.0-1.fc20.src.rpm
Description: KDBusAddons provides convenience classes on top of QtDBus, as well as an API to create KDED modules.
Fedora Account System Username: dvratil

Comment 1 Jan Grulich 2014-05-12 15:09:54 UTC
1) License should be LGPLv2+

And something from rpmlint:

2) W: shared-lib-calls-exit /usr/lib64/libKF5DBusAddons.so.4.99.0 exit.5 - this should be investigated whether it's not false positive warning.

3) W: incoherent-version-in-changelog 4.99.0 ['4.99.0-1.fc21', '4.99.0-1']

Comment 2 Daniel Vrátil 2014-05-13 08:36:37 UTC
Fixed license and changelog versioning in our git.

KDbusAddons indeed calls exit(), specifically in KDBusService class, when the service fails to connect to the bus during initialization. However exit() is invoked only when user does not specify KDBusService::NoExitOnFailure.

Comment 3 Jan Grulich 2014-05-13 09:17:35 UTC
Ok, can be approved.

Comment 4 Daniel Vrátil 2014-05-14 10:37:59 UTC
New Package SCM Request
=======================
Package Name: kf5-kdbusaddons
Short Description: KDE Frameworks Tier 1 addon with various classes on top of QtDBus
Owners: dvratil jgrulich kkofler rdieter ltinkl
Branches: 
InitialCC:

Comment 5 Gwyn Ciesla 2014-05-14 11:26:30 UTC
Git done (by process-git-requests).

Comment 6 Daniel Vrátil 2014-09-25 08:39:14 UTC
Package Change Request
=======================
Package Name: 
New Branches: f20
Owners: dvratil ltinkl jgrulich kkofler rdieter than

Comment 7 Daniel Vrátil 2014-09-25 08:48:33 UTC
Package Change Request
=======================
Package Name: kf5-kdbusaddons
New Branches: f20
Owners: dvratil ltinkl jgrulich kkofler rdieter than

Comment 8 Gwyn Ciesla 2014-09-25 12:17:32 UTC
Git done (by process-git-requests).


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