Created attachment 162110 [details]
svn diff for math/eigen3
Please find attached the trivial patch to version 3.2.6 bringing us up to date with the latest upstream release. See bugfixes since the 3.2.2 release here: eigen.tuxfamily.org/index.php?title=ChangeLog
Note that Eigen as per their homepage is released under MPL2 with few code exceptions under LGPL3+. Hence, I changed the LICENSE definitions to LGPL3 plus MPL2.
Created attachment 162167 [details]
fix plist and remove PORTREVISION
Additionally drop PORTREVISION, and fix plist
> Note that Eigen as per their homepage is released under MPL2 with few code exceptions under LGPL3+. Hence, I changed the LICENSE definitions to LGPL3 plus MPL2.
Looking at http://eigen.tuxfamily.org/index.php?title=Main_Page#License and COPYING.README, it doesn't seem to be that way. The files the wiki page mentions seem to be licensed under the LGPL2.1.
Re licensing: I changed this according to http://eigen.tuxfamily.org/index.php?title=Main_Page#License .
Which is the same page I linked to. It says "earlier versions were licensed under the LGPL3+", and then confusingly just refers to existing code not licensed under the MPL2 as "licensed under the LGPL". Looking at the third-party code it mentions, they all seem to be licensed under the LGPL21.
I see. So then LGPL2 and MPL2? Should I regenerate the patch with this as well as the plist and PORTREVISION changes by Tobias? Anything else before this can be committed?
LGPL21, not LGPL2. Basically, I believe the existing license list is correct. Please submit a new patch with this fix as well as Tobias's. If you have some spare cycles, I see that upstream has released v3.2.7 so you could update to that instead. If you don't have spare cycles, we can land 3.2.6 anyway.
Created attachment 163495 [details]
Eigen 3.6.7 update
Includes Tobias' changes, includes the update to 3.6.7 which required another plist change.
Ran this through a poudriere build on 9.3 for both i386 and amd64, a poudriere on 10.1/amd64 as well as my 11-CURRENT/amd64.
It passes portlint with one warning (missing ftp/http mirror in MASTER_SITES).
Sorry, it is late.
A commit references this bug:
Date: Wed Nov 25 13:25:55 UTC 2015
New revision: 402397
Update to 3.2.7.
Submitted by: email@example.com,
Tobias Berner <firstname.lastname@example.org>
Finally landed, thank you very much!