Bug 451505

Summary: Review Request: phonon - Multimedia framework api
Product: [Fedora] Fedora Reporter: Rex Dieter <rdieter>
Component: Package ReviewAssignee: Lorenzo Villani <lorenzo>
Status: CLOSED NEXTRELEASE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: fedora-package-review, kevin, notting
Target Milestone: ---Flags: lorenzo: fedora-review+
kevin: 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: 2008-06-17 01:05:23 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: 451655    

Description Rex Dieter 2008-06-15 02:14:39 UTC
Spec,SRPM URL: http://rdieter.fedorapeople.org/phonon/

Description: Multimedia framework api

Needed for future kde-4.1+ builds

scratch build:  http://koji.fedoraproject.org/koji/taskinfo?taskID=662394

Comment 1 Lorenzo Villani 2008-06-16 14:32:17 UTC
Package builds properly but:

- The patch listed in the .spec file is not applied (and rpmlint warns about it)
- phonon.i386: W: incoherent-version-in-changelog 4.2-0.1
4.2-0.1.20080614svn820634.fc9

Although it's not strictly required, documentation is missing from the -devel
subpackage:
- phonon-devel.i386: W: no-documentation


Comment 2 Rex Dieter 2008-06-16 19:06:08 UTC
The patch could be applied unconditionally, seems to fix a problem when building
on f9 with cmake-2.4.  f10/cmake-2.6 is fine.

Comment 3 Lorenzo Villani 2008-06-16 19:51:49 UTC
I don't see any MUST/blocker. The items listed above are just SHOULD/warnings so...

Approved

Comment 4 Rex Dieter 2008-06-16 20:09:45 UTC
New Package CVS Request
=======================
Package Name: phonon
Short Description: Multimedia framework api
Owners: rdieter
Branches: F-9
InitialCC:
Cvsextras Commits: yes


Comment 5 Dennis Gilmore 2008-06-16 20:46:59 UTC
CVS Done

Comment 6 Rex Dieter 2008-06-17 01:05:23 UTC
imported, built for rawhide.

Comment 7 Kevin Kofler 2008-07-23 14:37:39 UTC
Package Change Request
======================
Package Name: phonon
New Branches: F-8

Comment 8 Kevin Fenzi 2008-07-23 16:00:21 UTC
cvs done.