Bug 236422

Summary: Review Request: clutter-gst - ClutterMedia interface to GStreamer
Product: [Fedora] Fedora Reporter: Allisson Azevedo <allisson>
Component: Package ReviewAssignee: Mamoru TASAKA <mtasaka>
Status: CLOSED NEXTRELEASE QA Contact: Fedora Package Reviews List <fedora-package-review>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideFlags: mtasaka: fedora-review+
wtogami: fedora-cvs+
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2007-05-15 10:56:46 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:

Description Allisson Azevedo 2007-04-13 19:52:23 UTC
Spec URL: http://fedora.allisson.eti.br/clutter-gst/clutter-gst.spec
SRPM URL: http://fedora.allisson.eti.br/clutter-gst/clutter-gst-0.1.1-1.src.rpm

Description: This package contains a video texture actor and an audio player object for use with clutter

Comment 1 Mamoru TASAKA 2007-05-12 15:59:22 UTC
Will review.

Comment 2 Mamoru TASAKA 2007-05-13 14:58:21 UTC
For 0.1.1-1:

* Requires/BuildRequires
  - clutter-devel requires gtk2-devel and then glib2-devel,
    so in this case explicit requirement of gtk2-devel or
    glib2-devel is not needed.

  - clutter-gst-devel requires (not "BuildRequire"s) 
    gstreamer-plugins-base-devel which is pulled by
    "Requires: gstreamer-plugins-base-0.10" in pkgconfig .pc file.

* Timestamps
  - Similarly to clutter-gtk, please add 'INSTALL=%{__install} -p"
    to "make install".

* Documentation
  - INSTALL is not needed.

* License
  - This is LGPL.

Comment 3 Allisson Azevedo 2007-05-14 01:59:49 UTC
Update package:

Spec URL: http://fedora.allisson.eti.br/clutter-gst/clutter-gst.spec
SRPM URL: http://fedora.allisson.eti.br/clutter-gst/clutter-gst-0.1.1-2.src.rpm

Changelog:

- INSTALL removed from docs
- fix make install for keeping timestamps
- changed license for LGPL
- Fix requires/buildrequires

Comment 4 Mamoru TASAKA 2007-05-14 15:31:18 UTC
Okay.
---------------------------------------------
  This package (clutter-gst) is APPROVED by me
---------------------------------------------

Comment 5 Allisson Azevedo 2007-05-14 18:29:13 UTC
New Package CVS Request
=======================
Package Name: clutter-gst
Short Description: ClutterMedia interface to GStreamer
Owners: allisson
Branches: FC-6