Bug 193584

Summary:	Review Request: nucleo - Toolkit for exploring new uses of video
Product:	[Fedora] Fedora	Reporter:	Christopher Stone <chris.stone>
Component:	Package Review	Assignee:	Luya Tshimbalanga <luya_tfz>
Status:	CLOSED NEXTRELEASE	QA Contact:	Fedora Package Reviews List <fedora-package-review>
Severity:	medium	Docs Contact:
Priority:	medium
Version:	rawhide	CC:	fedorajim, wtogami
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:	2006-06-01 20:26: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:
Bug Blocks:	163779, 177080

Description Christopher Stone 2006-05-30 19:34:24 UTC

Spec URL: http://tkmame.retrogames.com/fedora-extras/nucleo.spec
SRPM URL: http://tkmame.retrogames.com/fedora-extras/nucleo-0.5-2.src.rpm

Description:
Nucleo is a toolkit for exploring new uses of video and new human-computer
interaction techniques.

This should obsolete bug #177081

Comment 1 Luya Tshimbalanga 2006-05-30 22:51:36 UTC

- Spec files followed the Packaging Guideline
http://fedoraproject.org/wiki/Packaging/Guidelines and the Package Naming
Guideline http://fedoraproject.org/wiki/Packaging/NamingGuidelines
- sha1sum and md5sum from SRPM match the source package from the original website :
http://insitu.lri.fr/~chapuis/software/metisse/
- The built package passed the rpmlint check
- The license from the package is LGPL 
- In accordance, the spec and srpms passed the review  and should be added in
the cvs extras

Comment 2 Luya Tshimbalanga 2006-05-30 22:57:22 UTC

*** Bug 177081 has been marked as a duplicate of this bug. ***

Comment 3 Warren Togami 2006-06-01 14:42:19 UTC

%exclude %{_libdir}/*.la

It is preferable that you delete this file during %build instead of exclude, but
otherwise it looks OK.

You should remove the INSTALL file.  It is only generic install instructions and
not useful to include in the binary RPM.