Bug 830362 (kscd)

Summary: Review Request: kscd - CD Player
Product: [Fedora] Fedora Reporter: Rex Dieter <rdieter>
Component: Package ReviewAssignee: Jaroslav Reznik <jreznik>
Status: CLOSED RAWHIDE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: jreznik, kevin, notting, package-review
Target Milestone: ---Flags: jreznik: 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: 2012-06-20 16:42:22 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: 827008, 829393, 830360    
Bug Blocks: 656997    

Description Rex Dieter 2012-06-09 00:42:03 UTC
Spec URL: http://rdieter.fedorapeople.org/rpms/kdemm/kscd.spec
SRPM URL: http://rdieter.fedorapeople.org/rpms/kdemm/kscd-4.8.90-1.fc17.src.rpm
Description: CD Player
Fedora Account System Username: rdieter

New package from upstream splitting the once-monolithic kdemultimedia tarball (and why this one includes Obsoletes)

Comment 1 Rex Dieter 2012-06-14 18:43:34 UTC
Spec URL: http://rdieter.fedorapeople.org/rpms/kdemm/kscd.spec
SRPM URL: http://rdieter.fedorapeople.org/rpms/kdemm/kscd-4.8.90-2.fc17.src.rpm

%changelog
* Wed Jun 13 2012 Rex Dieter <rdieter> 4.8.90-2
- +%%doc ChangeLog README TODO

Comment 2 Jaroslav Reznik 2012-06-19 16:26:29 UTC
Name: ok
Summary: ok
License: ok, GPLv2+
URL: I'd use this one http://www.kde.org/applications/multimedia/kscd/ (more user friendly)
Sources: ok (md5sum fd334bf6dec27e8b6706e196638a2bef)
BuildRequires: ok
Requires: ok
Obsoletes/Provides: ok, kdemultimedia-kscd split
Description: ok (maybe more descriptive "KsCD is a small, fast, CDDB enabled audio CD player.")
Build: ok
Macros used consistently: ok
Desktop file validation: ok
Icon cache update: ok
Docs: ok

rpmlint kscd-4.8.90-2.fc17.src.rpm 
kscd.src:53: W: macro-in-comment %{name}
1 packages and 0 specfiles checked; 0 errors, 1 warnings.

#find_lang %{name} --with-kde --all-name
are translations in kde-l10n or not?

Comment 3 Rex Dieter 2012-06-19 16:49:09 UTC
the find_lang call here are used to pick up the khelpcenter handbooks, but kscd doesn't have one.  I probably should've just removed it, rather than comment it out

Comment 4 Rex Dieter 2012-06-19 16:49:42 UTC
But otherwise, yes, gettext translations are in kde-l10n (or should be)

Comment 5 Rex Dieter 2012-06-19 17:38:33 UTC
Spec URL: http://rdieter.fedorapeople.org/rpms/kdemm/kscd.spec
SRPM URL: http://rdieter.fedorapeople.org/rpms/kdemm/kscd-4.8.90-3.fc17.src.rpm

%changelog
* Tue Jun 19 2012 Rex Dieter <rdieter> 4.8.90-3
- update URL, %%description

Comment 6 Jaroslav Reznik 2012-06-20 14:27:27 UTC
Thanks Rex,
APPROVED.

Comment 7 Rex Dieter 2012-06-20 15:12:40 UTC
thanks.

New Package SCM Request
=======================
Package Name: kscd
Short Description: CD player
Owners: than,rdieter,jreznik,rnovacek,ltinkl,kkofler
Branches: f17
InitialCC:

Comment 8 Gwyn Ciesla 2012-06-20 15:16:02 UTC
Git done (by process-git-requests).

Comment 9 Rex Dieter 2012-06-20 16:42:22 UTC
imported