Bug 549980

Summary: Review Request: jama - C++ matrix templates
Product: [Fedora] Fedora Reporter: Matt Chan <talcite>
Component: Package ReviewAssignee: Peter Lemenkov <lemenkov>
Status: CLOSED DEFERRED QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: low    
Version: rawhideCC: fedora-package-review, i, lemenkov, notting, talcite
Target Milestone: ---Flags: lemenkov: fedora‑review+
dennis: 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: 2010-10-28 03:55:47 EDT Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---
Bug Depends On:    
Bug Blocks: 201449, 518949    

Description Matt Chan 2009-12-23 01:26:39 EST
This package is being requested as a part of the brlcad package. It is being abstracted to clean out external libraries.

Spec URL: http://mattchan.homelinux.net:55555/jama/jama-devel.spec
SRPM URL: http://mattchan.homelinux.net:55555/jama/jama-devel-1.2.5-1.fc12.src.rpm
Description: A set of C++ headers implementing the Java Matrix Library. Contains templates for classical matrix algebra and for a series of matrix decompositions such as LU, QR, SVD, Cholesky methods.
Comment 1 Matt Chan 2009-12-23 23:49:45 EST
updated spec and SRPM to BuildArch: noarch since package consists of only header files.

Spec URL: http://mattchan.homelinux.net:55555/jama/jama-devel.spec
SRPM URL: http://mattchan.homelinux.net:55555/jama/jama-devel-1.2.5-2.fc12.src.rpm
Comment 2 Peter Lemenkov 2009-12-24 14:32:28 EST
Unblocking FE-NEEDSPONSOR since I just sponsored Matt.
Comment 3 Peter Lemenkov 2009-12-24 14:34:24 EST
I'll review it.
Comment 4 Matt Chan 2009-12-26 03:54:51 EST
Updated package name to jama from jama-devel and included the use of a subpackage from feedback on the tnt package. Also cleaned up the %install section. 

SPEC: http://mattchan.homelinux.net:55555/jama/jama.spec
SRPM: http://mattchan.homelinux.net:55555/jama/jama-1.2.5-2.fc12.src.rpm

Matt
Comment 5 Peter Lemenkov 2009-12-26 05:49:43 EST
REVIEW:

+ rpmlint is not silent, but his the only warning may be omitted.

[petro@Sulaco SPECS]$ rpmlint ../RPMS/noarch/jama-devel-1.2.5-2.fc12.noarch.rpm 
jama-devel.noarch: W: no-documentation
1 packages and 0 specfiles checked; 0 errors, 1 warnings.
[petro@Sulaco SPECS]$

+ The package is named according to the  Package Naming Guidelines.
+ The spec file name matches the base package %{name}, in the format %{name}.spec.
+ The package meets the Packaging Guidelines.
+ The package is licensed with a Fedora approved license and meets the Licensing Guidelines.
+ The License field in the package spec file matches the actual license.
0 Upstream doesn't provide the file file, containing the text of the license(s) for the package.
+ The spec file is written in American English.
+ The spec file for the package is legible.
+ The sources used to build the package, match the upstream source, as provided in the spec URL.

[petro@Sulaco SOURCES]$ sha256sum jama125.zip*
7432bf51d7101f63d5880421f940f64cea124f1460ff2fcfb6226e6744d1360c  jama125.zip
7432bf51d7101f63d5880421f940f64cea124f1460ff2fcfb6226e6744d1360c  jama125.zip.1
[petro@Sulaco SOURCES]$ 

+ The package successfully compiles and builds into binary rpms on at least one primary architecture.
+ All build dependencies are listed in BuildRequires.
0 No need to handle locales.
0 No shared library files.
+ The package does NOT bundle copies of system libraries.
+ The package is not designed to be relocatable.
+ The package owns all directories that it creates.
+ The package does not list a file more than once in the spec file's %files listings.
+ Permissions on files are set properly.
+ The package has a %clean section, which contains rm -rf %{buildroot} (or $RPM_BUILD_ROOT).
+ The package consistently uses macros.
+ The package contains code, or permissible content.
0 No extremely large documentation files.
0 The package doesn't contain %doc files.
+ C header files packaged in devel sub-package.
0 No static libraries.
0 No pkgconfig(.pc) files.
0 The package doesn't contain library files with a suffix (e.g. libfoo.so.1.1).
+ The package does NOT contain any .la libtool archives.
0 Not a GUI application.
+ The package does not own files or directories already owned by other packages.
+ At the beginning of %install, the package runs rm -rf %{buildroot} (or $RPM_BUILD_ROOT).
+ All filenames in rpm packages are valid UTF-8.

APPROVED.
Comment 6 Peter Lemenkov 2010-01-26 06:33:53 EST
Ping, Matt!
Comment 7 Peter Lemenkov 2010-04-20 05:30:33 EDT
Ping, again.
Comment 8 Matt Chan 2010-05-17 23:49:36 EDT
New Package CVS Request
=======================
Package Name: jama
Short Description: A collection of C++ matrix templates
Owners: talcite
Branches: F-12 F-13 EL-5
InitialCC:
Comment 9 Dennis Gilmore 2010-05-18 14:38:26 EDT
CVS Done 

Does this need a EL-6 branch also?
Comment 10 Matt Chan 2010-05-18 15:34:31 EDT
Ah! The wiki page didn't mention that EL-6 was an option. Yes, please create one.

Package Change Request
======================
Package Name: jama
New Branches: EL-6
Owners: talcite
Comment 11 Dennis Gilmore 2010-05-18 16:50:08 EDT
CVS Done
Comment 12 Peter Lemenkov 2010-06-01 14:22:43 EDT
Ping, Matt.
Comment 13 Peter Lemenkov 2010-10-18 01:55:08 EDT
Ping, again.
Comment 14 Peter Lemenkov 2010-10-25 07:10:59 EDT
Last ping. I'll close this ticket next week if there will be no answer.
Comment 15 Christopher Meng 2013-11-28 06:29:36 EST
ping again.