Bug 1278671 (octave-metch)

Summary: Review Request: octave-metch - Mesh/volume registration toolbox
Product: [Fedora] Fedora Reporter: Igor Gnatenko <ignatenko>
Component: Package ReviewAssignee: Zbigniew Jędrzejewski-Szmek <zbyszek>
Status: CLOSED ERRATA QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: package-review, zbyszek
Target Milestone: ---Flags: zbyszek: fedora-review+
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2015-12-15 13:22:56 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: 1276941    

Description Igor Gnatenko 2015-11-06 06:38:54 UTC
Spec URL: https://ignatenkobrain.fedorapeople.org/neurofedora/octave-metch.spec
SRPM URL: https://ignatenkobrain.fedorapeople.org/neurofedora/octave-metch-0.5.0-1.fc24.src.rpm
Description:
Matlab/Octave-based mesh/volume registration toolbox. It provides
straightforward functions to register point clouds (or surfaces) to a
triangular/cubic surface mesh by calculating an optimal affine transformation
(in terms of matrix A for scaling and rotation, and b for translation). It
also allows one to project a point cloud onto the surface using surface norms
and guarantee the conformity of the points to the surface. At this point, metch
can only perform rigid-body registration in terms of a linear transformation. 
Fedora Account System Username: ignatenkobrain

Comment 1 Zbigniew Jędrzejewski-Szmek 2015-12-02 00:23:42 UTC
- license is OK
- license file is present, %license is used
- latest version
- standard scriptlets
- builds and installs fine
- summary and description are OK
- package name is OK

Rpmlint
-------
Checking: octave-metch-0.5.0-1.fc24.noarch.rpm
          octave-metch-0.5.0-1.fc24.src.rpm
octave-metch.noarch: W: spelling-error %description -l en_US Matlab -> Mat lab, Mat-lab, Mazatlan
octave-metch.noarch: W: spelling-error %description -l en_US affine -> caffeine, fine
octave-metch.src: W: spelling-error %description -l en_US Matlab -> Mat lab, Mat-lab, Mazatlan
octave-metch.src: W: spelling-error %description -l en_US affine -> caffeine, fine
2 packages and 0 specfiles checked; 3 errors, 5 warnings.

All OK.

Rpmlint (installed packages)
----------------------------
octave-metch.noarch: E: script-without-shebang /usr/share/octave/packages/metch-0.5.0/trisurfnorm.m
octave-metch.noarch: E: script-without-shebang /usr/share/octave/packages/metch-0.5.0/nodesurfnorm.m
octave-metch.noarch: E: script-without-shebang /usr/share/octave/packages/metch-0.5.0/dist2surf.m
Those three have spurious +x permission. Please remove.

octave-metch.noarch: W: dangerous-command-in-%preun mv
OK, comes from a standard scriptlet.

1 packages and 0 specfiles checked; 3 errors, 1 warnings.

Requires
--------
octave-metch (rpmlib, GLIBC filtered):
    /bin/sh
    octave

Provides
--------
octave-metch:
    octave-metch

Please fix the permissions. Package is APPROVED.

Comment 2 Till Maas 2015-12-06 10:26:31 UTC
Package request has been approved: https://admin.fedoraproject.org/pkgdb/package/octave-metch

Comment 3 Fedora Update System 2015-12-06 11:50:54 UTC
octave-metch-0.5.0-1.fc23 has been submitted as an update to Fedora 23. https://bodhi.fedoraproject.org/updates/FEDORA-2015-84332ef5c6

Comment 4 Fedora Update System 2015-12-06 23:51:20 UTC
octave-metch-0.5.0-1.fc23 has been pushed to the Fedora 23 testing repository. If problems still persist, please make note of it in this bug report.
If you want to test the update, you can install it with
$ su -c 'dnf --enablerepo=updates-testing update octave-metch'
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2015-84332ef5c6

Comment 5 Fedora Update System 2015-12-15 13:22:54 UTC
octave-metch-0.5.0-1.fc23 has been pushed to the Fedora 23 stable repository. If problems still persist, please make note of it in this bug report.