Bug 917145

Summary:	eigen3 3.1.2 is available
Product:	[Fedora] Fedora	Reporter:	Sandro Mani <manisandro>
Component:	eigen3	Assignee:	Rich Mattes <richmattes>
Status:	CLOSED RAWHIDE	QA Contact:	Fedora Extras Quality Assurance <extras-qa>
Severity:	unspecified	Docs Contact:
Priority:	unspecified
Version:	rawhide	CC:	c.david86, richmattes
Target Milestone:	---
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2013-03-21 02:24:01 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:

Description Sandro Mani 2013-03-01 20:28:34 UTC

Description of problem:
eigen3 3.1.2 was released 05.11.2012, see

http://eigen.tuxfamily.org/index.php?title=Main_Page

Note: eigen3 3.1.2 is now licensed MPL2.

Comment 1 Sandro Mani 2013-03-01 20:40:22 UTC

Actually, concerning licensing, the COPYING.README states

Eigen is primarily MPL2 licensed. See COPYING.MPL2 and these links:
  http://www.mozilla.org/MPL/2.0/
  http://www.mozilla.org/MPL/2.0/FAQ.html

Some files contain third-party code under BSD or LGPL licenses, whence the other
COPYING.* files here.

All the LGPL code is either LGPL 2.1-only, or LGPL 2.1-or-later.
For this reason, the COPYING.LGPL file contains the LGPL 2.1 text.


So License should be "MPL2 and LGPLv2+ and BSD"

Comment 2 Rich Mattes 2013-03-04 19:27:25 UTC

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

Comment 3 Rich Mattes 2013-03-04 19:28:11 UTC

Thanks, I'll fix the licensing and roll the new eigen3 into rawhide soon.

Comment 4 Sandro Mani 2013-03-05 10:39:37 UTC

Just a note: since I've just become maintainer for mingw-eigen3, if you want, I could also help co-maintain eigen3, to allow for easier synchronization between the two packages.

(Btw, MPL2 should be MPLv2.0 to conform with the guidelines)

Comment 5 Rich Mattes 2013-03-06 00:24:46 UTC

Sure, I don't mind if you want to help co-maintain eigen3.  I'm working on getting 3.1.2 ready now.  

I'm curious as to why a separate mingw version of eigen3 is required though; eigen3 is just a header library, so the system installed version should be usable by mingw's compiler.  Unless it's just a question of installation location, but that could be solved by symlinking probably.

Comment 6 Sandro Mani 2013-03-06 09:01:05 UTC

Concerning the mingw variant, I agree, the only interesting part of it is shipping the pkg-config file available in the mingw environment It's certainly the case that one could simply require eigen3 and symlink /usr/i686-w64-mingw{32,64}/sys-root/mingw/include/eigen3 to /usr/include/eigen3, if that is legal to do in a package.

Comment 7 Rich Mattes 2013-03-21 02:24:01 UTC

There's nothing in the guidelines that forbids making those types of symlinks that I'm aware of.  The mingw package is already reviewed though so I don't know if it's all that important.

I've updated to 3.1.2 in rawhide and sent an email to the devel list about the license change, so I think we're all good now.  If you still want to help co-maintain eigen3 go ahead and apply in pkgdb and I'll approve you.

Comment 8 Sandro Mani 2013-03-21 08:47:56 UTC

Thanks, I've added myself to the package for fedora-devel + branches. I'll ask on devel for opinions concerning the mingw package.

Comment 9 Sandro Mani 2013-03-21 09:03:52 UTC

(Actually, I think I'll just leave the mingw package as is. After other mingw packages duplicate the include headers as well, so the current approach is more consistent.)