Bug 1094329 (kf5-kcoreaddons) - Review Request: kf5-kcoreaddons - KDE Frameworks 5 Tier 1 addon with various classes on top of QtCore
Summary: Review Request: kf5-kcoreaddons - KDE Frameworks 5 Tier 1 addon with various ...
Keywords:
Status: CLOSED RAWHIDE
Alias: kf5-kcoreaddons
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:12 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:05:53 UTC
Type: ---
Embargoed:
jgrulich: fedora-review+
gwync: fedora-cvs+


Attachments (Terms of Use)

Description Daniel Vrátil 2014-05-05 13:12:49 UTC
Spec URL: http://dvratil.fedorapeople.org/kf5/review/kf5-kcoreaddons.spec
SRPM URL: http://dvratil.fedorapeople.org/kf5/review/kf5-kcoreaddons-4.99.0-1.fc20.src.rpm
Description: KCoreAddons provides classes built on top of QtCore to perform various tasks such as manipulating mime types, autosaving files, creating backup files, generating random sequences, performing text manipulations such as macro
replacement, accessing user information and many more.
Fedora Account System Username: dvratil

Comment 1 Jan Grulich 2014-05-06 16:02:34 UTC
Build fine on Rawhide. License should be GPLv2+ and LGPLv2+.

Rpmlint shows errors:
kf5-kcoreaddons.x86_64: W: shared-lib-calls-exit /usr/lib64/libKF5CoreAddons.so.4.99.0 exit.5

Not sure whether this one is false positive.

kf5-kcoreaddons.x86_64: E: incorrect-locale-subdir /usr/share/locale/x-test/LC_MESSAGES/kcoreaddons5_qt.qm

This should be false positive.

Comment 2 Daniel Vrátil 2014-05-13 08:25:19 UTC
Fixed license in our git.

exit(EXIT_SUCCESS) is indeed called from KCoreAddons, specifically from KAboutData class. KAboutData invokes exit() when application is started with --author or --license argument. At this point, calling exit() is expected behavior as the application prints author or license and terminates (also, it's the same as in kdelibs4).

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

Comment 4 Daniel Vrátil 2014-05-13 13:07:01 UTC
New Package SCM Request
=======================
Package Name: kf5-kcoreaddons
Short Description: KDE Frameworks 5 Tier 1 addon with various classes on top of QtCore
Owners: dvratil jgrulich ltinkl rdieter kkofler
Branches: 
InitialCC:

Comment 5 Gwyn Ciesla 2014-05-13 15:02:51 UTC
Git done (by process-git-requests).

Comment 6 Daniel Vrátil 2014-09-25 08:39:04 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:12 UTC
Package Change Request
=======================
Package Name: kf5-kcoreaddons
New Branches: f20
Owners: dvratil ltinkl jgrulich kkofler rdieter than

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


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