Bug 451505 - Review Request: phonon - Multimedia framework api
Review Request: phonon - Multimedia framework api
Status: CLOSED NEXTRELEASE
Product: Fedora
Classification: Fedora
Component: Package Review (Show other bugs)
rawhide
All Linux
medium Severity medium
: ---
: ---
Assigned To: Lorenzo Villani
Fedora Extras Quality Assurance
:
Depends On:
Blocks: 451655
  Show dependency treegraph
 
Reported: 2008-06-14 22:14 EDT by Rex Dieter
Modified: 2008-07-23 12:00 EDT (History)
3 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2008-06-16 21:05:23 EDT
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---
lorenzo: fedora‑review+
kevin: fedora‑cvs+


Attachments (Terms of Use)

  None (edit)
Description Rex Dieter 2008-06-14 22:14:39 EDT
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 10:32:17 EDT
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 15:06:08 EDT
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 15:51:49 EDT
I don't see any MUST/blocker. The items listed above are just SHOULD/warnings so...

Approved
Comment 4 Rex Dieter 2008-06-16 16:09:45 EDT
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 16:46:59 EDT
CVS Done
Comment 6 Rex Dieter 2008-06-16 21:05:23 EDT
imported, built for rawhide.
Comment 7 Kevin Kofler 2008-07-23 10:37:39 EDT
Package Change Request
======================
Package Name: phonon
New Branches: F-8
Comment 8 Kevin Fenzi 2008-07-23 12:00:21 EDT
cvs done.

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