Bug 592074

Summary: Review Request: R-SparseM - Sparse linear algebra
Product: [Fedora] Fedora Reporter: Sandro Mathys <sandro>
Component: Package ReviewAssignee: Pierre-YvesChibon <pingou>
Status: CLOSED WONTFIX QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: fedora-package-review, notting, pingou, tcallawa
Target Milestone: ---Flags: sandro: 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: 2010-09-22 09:30:24 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: 182235    

Description Sandro Mathys 2010-05-13 15:27:38 EDT
Spec URL: http://red.fedorapeople.org/SRPMS/R-SparseM.spec
SRPM URL: http://red.fedorapeople.org/SRPMS/R-SparseM-0.85-1.fc12.src.rpm
Description:
Basic linear algebra for sparse matrices.

rpmlint {SPECS,SRPMS,RPMS/i686}/R-SparseM*
3 packages and 1 specfiles checked; 0 errors, 0 warnings.
Comment 1 Pierre-YvesChibon 2010-06-28 02:37:09 EDT
* Latest version packaged
62ffeff7a9739019b3c42e3eab31f1a89a4d102d  SparseM_0.85.tar.gz
62ffeff7a9739019b3c42e3eab31f1a89a4d102d  ~/rpmbuild/SOURCES/SparseM_0.85.tar.gz
* sha1sum are equals
* All required Requires are present
* All required BuildRequires are present
* The macro %check is present
* There is 1 %dir
  %dir is OK
* There is 5 %doc
* More element than expected in %doc
   LICENSE ChangeLog
* The rpm installed in _libdir
* The rpm uses %{_libdir} and is arch
* The rpm seems to have the required element in %install
* Build properly under 2.6.33.5-124.fc13.x86_64
* rpmlint:
  3 packages and 0 specfiles checked; 0 errors, 0 warnings.

  Scratch build on koji for target f14
* Koji build: http://koji.fedoraproject.org/koji/taskinfo?taskID=2276476
   0 free  0 open  3 done  0 failed
   2276476 build (dist-f14, R-SparseM-0.85-1.fc12.src.rpm) completed successfully


- Build produces quite a number of warnings
* %check is present
* All checks are passed successfully
* License GPLv2+ is compatible with Fedora
! The license file mention different licenses (and provides a incorrect link)
I will ask for information regarding the license before approve this package.
Comment 2 Tom "spot" Callaway 2010-06-28 08:24:07 EDT
The PCx license (on cholesky.f) is non-free, because of the commercial use restriction.

I made Paul Betten aware of the licensing issue on September 2, 2009, and asked him if it would be possible to relicense it to something Free and GPL compatible. He replied that he would look into the issue, but I have never received any further communication from Mr. Betten.

Blocking this on FE-Legal.
Comment 3 Pierre-YvesChibon 2010-06-30 02:09:59 EDT
Is it something worth mentioning to CRAN ? Or is CRAN actually allowed to distribute this package ?
Comment 4 Sandro Mathys 2010-07-12 15:36:11 EDT
I wrote Paul Betten a mail, maybe a second poke after the one of spot helps - you never know. No idea what else we could do here.
Comment 5 Sandro Mathys 2010-07-12 18:08:17 EDT
Paul Betten stated that PCx is commercially licensed and he can't carve out a routine (cholesky.f).
Comment 6 Pierre-YvesChibon 2010-08-30 10:14:28 EDT
I believe we should close this bug as the license is a blocker for Fedora. Sandro, shall I do it ?
Comment 7 Pierre-YvesChibon 2010-09-21 14:13:05 EDT
ping ?
Comment 8 Sandro Mathys 2010-09-22 09:30:24 EDT
cholesky.f is nonfree, therefore this can't be included in Fedora.