[Bug 753453] Review Request: Field3D - Library for storing voxel data

[bugzilla at redhat.com]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=753453

Mo Morsi <mmorsi at redhat.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
               Flag|fedora-review?              |fedora-review+

--- Comment #4 from Mo Morsi <mmorsi at redhat.com> 2011-12-13 18:27:15 EST ---
As promised, reviewing this package

* rpmlint looks good

$ rpmlint /home/mmorsi/rpmbuild/RPMS/x86_64/Field3D-1.2.1-1.fc15.x86_64.rpm
/home/mmorsi/rpmbuild/RPMS/x86_64/Field3D-devel-1.2.1-1.fc15.x86_64.rpm
/home/mmorsi/rpmbuild/RPMS/x86_64/Field3D-debuginfo-1.2.1-1.fc15.x86_64.rpm
rpmbuild/SRPMS/Field3D-1.2.1-1.fc15.src.rpm 
Field3D.x86_64: W: spelling-error Summary(en_US) voxel -> vowel
Field3D.x86_64: W: spelling-error %description -l en_US voxel -> vowel
Field3D.src: W: spelling-error Summary(en_US) voxel -> vowel
Field3D.src: W: spelling-error %description -l en_US voxel -> vowel
Field3D.src: W: invalid-url Source0:
imageworks-Field3D-v1.2.1-0-g0790dc2.tar.gz
4 packages and 0 specfiles checked; 0 errors, 5 warnings.

* koji is green:
http://koji.fedoraproject.org/koji/taskinfo?taskID=3583433

* package performs are required as verified via the test suite and running the
f3dinfo --help

* package matches upstream source
$ md5sum imageworks-Field3D-v1.2.1-0-g0790dc2.tar.gz
~/Downloads/imageworks-Field3D-v1.2.1-0-g0790dc2.tar.gz 
9373b2a11df85a2b5cc4ab1f793493d4  imageworks-Field3D-v1.2.1-0-g0790dc2.tar.gz
9373b2a11df85a2b5cc4ab1f793493d4 
/home/mmorsi/Downloads/imageworks-Field3D-v1.2.1-0-g0790dc2.tar.gz


The package looks good, and complies to packaging guidelines. Good job.


---------------
Field3D is APPROVED
---------------

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.