-
Notifications
You must be signed in to change notification settings - Fork 26
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
License #1
Comments
Hi, The release v1.0.0 is imminent. The license says
@andreadelprete: do you want to add anything ? |
The original license is coming from people of Eigen. Please copy/past where it is required. |
I think @jmirabel may provide some hints about the original license |
Here are the pieces of information I could gather:
|
So the core code of this module should be LGPL and not BSD as currently mentioned. |
I think it should be GPL, except if Gael Guennebaud (and maybe Benjamin Stephens) agrees to change this to LGPL. |
Very good job backtracking the origin of this piece of code @jmirabel ! |
Thanks for that job :D From the link in #2, it looks like Gael Guennebaud latest version is LGPL, so I think that we can use that. |
I think you can assume Gael Guennebaud has already accepted the LGPL licence. |
According to GNU, for the LGPL (and their other licenses), in addition of the main file, we have to put a license notice in each and every file. I hate that, but will do it. |
Fixed and released. Thank you all on this issue :) |
Thanks @nim65s :) |
The original mention of the license of the code is largely missing.
The code is a copy paster or former code by Andrea.
This should be fixed before any formal release I would say.
The text was updated successfully, but these errors were encountered: