Bug 1214299 (kf5-bluez-qt)

Summary: Review Request: kf5-bluez-qt - A Qt wrapper for Bluez
Product: [Fedora] Fedora Reporter: Daniel Vrátil <dvratil>
Component: Package ReviewAssignee: Jan Grulich <jgrulich>
Status: CLOSED RAWHIDE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: jeischma, jgrulich, kevin, package-review, rdieter
Target Milestone: ---Flags: jgrulich: fedora-review+
gwync: fedora-cvs+
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2015-05-05 15:43:57 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 656997, 1086148, 1135103    

Description Daniel Vrátil 2015-04-22 11:47:51 UTC
Spec URL: https://dvratil.fedorapeople.org/plasma5/review/kf5-bluez-qt.spec
SRPM URL: https://dvratil.fedorapeople.org/plasma5/review/kf5-bluez-qt-5.2.95-1.fc21.src.rpm
Description: A Qt wrapper for Bluez
Fedora Account System Username: dvratil

kf5-bluez-qt is currently released as part of Plasma 5 (hence the 5.2.95 version), but according to upstream the plan is to move it to Frameworks around 5.11, hence the name.

Comment 1 Daniel Vrátil 2015-04-22 11:51:37 UTC
Question: should this deprecate libbluedevil, since bluedevil now depends on this instead of libbluedevil?

Comment 2 Jan Grulich 2015-04-22 12:49:57 UTC
1) I think it should deprecate libbluedevil, there won't be probably any new release. Unless you want to keep KDE 4 version from some reason.

2) #%doc COPYING.LIB README.md → Why is this commented out?

Except of above mentioned issues it looks good. License is correct, scriptlets are also correct.

Comment 3 Daniel Vrátil 2015-04-22 13:05:12 UTC
Spec URL: https://dvratil.fedorapeople.org/plasma5/review/kf5-bluez-qt.spec
SRPM URL: https://dvratil.fedorapeople.org/plasma5/review/kf5-bluez-qt-5.2.95-2.fc21.src.rpm

* Wed Apr 22 2015 Daniel Vrátil <dvratil> - 5.2.95-2
- install %%doc
- Obsoletes: libbluedevil

Comment 4 Jan Grulich 2015-04-22 13:25:30 UTC
Looks good now, approved.

Comment 5 Daniel Vrátil 2015-04-22 18:45:30 UTC
New Package SCM Request
=======================
Package Name: pkgname
Short Description: summary of package
Upstream URL: http://pkgname-project.org/pkgname
Owners: dvratil ltinkl jgrulich rdieter kkofler
Branches: f22
InitialCC: group::kde-sig

Comment 6 Gwyn Ciesla 2015-04-22 19:44:04 UTC
WARNING: Requested package name pkgname doesn't match bug summary
kf5-bluez-qt

Comment 7 Daniel Vrátil 2015-04-23 12:03:12 UTC
New Package SCM Request
=======================
Package Name: kf5-bluez-qt
Short Description: A Qt wrapper for Bluez
Upstream URL: https://projects.kde.org/projects/kde/workspace/bluez-qt
Owners: dvratil ltinkl jgrulich rdieter kkofler
Branches: f22
InitialCC: group::kde-sig

Comment 8 Gwyn Ciesla 2015-04-23 12:40:59 UTC
Git done (by process-git-requests).

Comment 9 Rex Dieter 2015-05-05 15:43:57 UTC
imported