Bug 461431

Summary:	Review Request: clutter-cairomm - C++ wrapper for clutter-cairo library
Product:	[Fedora] Fedora	Reporter:	Denis Leroy <denis>
Component:	Package Review	Assignee:	Mamoru TASAKA <mtasaka>
Status:	CLOSED NEXTRELEASE	QA Contact:	Fedora Extras Quality Assurance <extras-qa>
Severity:	medium	Docs Contact:
Priority:	medium
Version:	rawhide	CC:	fedora-package-review, notting
Target Milestone:	---	Flags:	mtasaka: 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-09-15 08:43:27 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 Denis Leroy 2008-09-07 22:37:16 UTC

Spec URL: http://www.poolshark.org/src/clutter-cairomm.spec
SRPM URL: http://www.poolshark.org/src/clutter-cairomm-0.7.2-1.fc10.src.rpm

Description: 
Clutter-cairomm is a C++ wrapper library for clutter-cairo: a library
that allows clutter to be embedded in cairo applications. Clutter is
an OpenGL-based library for creating fast, visually rich and animated
graphical user interfaces.

Notes to reviewer: this library used to be packaged as part of cluttermm, but was recently forked from the cluttermm tarball (along with clutter-gtkmm), to match the same subpackages as the corresponding C libraries (clutter, clutter-cairo and clutter-gtk).

Comment 1 Mamoru TASAKA 2008-09-12 07:26:28 UTC

Okay.

! Please consider to use
  make install DESTDIR=$RPM_BUILD_ROOT INSTALL="install -p"
  to keep timestamps on installed files (for this package, especially
  for header files).
  This method usually works for recent autotools based
  Makefiles.

! Requires: clutter-devel seems redundant under the existence
  of "Requires: cluttermm-devel", however this is not a blocker.

-----------------------------------------------------------------------
    This package (clutter-cairomm) is APPROVED by mtasaka
-----------------------------------------------------------------------

Comment 2 Mamoru TASAKA 2008-09-12 07:30:49 UTC

Just a note:
successful scratch build:
http://koji.fedoraproject.org/koji/taskinfo?taskID=822085

Comment 3 Denis Leroy 2008-09-12 10:44:07 UTC

Thanks, I added your suggested fixes.

New Package CVS Request
=======================
Package Name: clutter-cairomm
Short Description: C++ wrapper for clutter-cairo library
Owners: denis
Branches: F-9
InitialCC:

Comment 4 Kevin Fenzi 2008-09-14 22:27:56 UTC

cvs done.

Comment 5 Denis Leroy 2008-09-15 08:43:27 UTC

Thanks.