Skip to content
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 change #7

Closed
xhub opened this issue Jan 21, 2016 · 9 comments
Closed

License change #7

xhub opened this issue Jan 21, 2016 · 9 comments
Assignees
Milestone

Comments

@xhub
Copy link
Member

xhub commented Jan 21, 2016

Need to update almost all files.

@xhub xhub added this to the 3.8 milestone Jan 21, 2016
@vacary
Copy link
Member

vacary commented Mar 11, 2016

Just an update of this issue, are we ready to change to LGPL ?

@vacary
Copy link
Member

vacary commented Mar 14, 2016

Why not Apache 2.0 rather LGPL ?

@xhub
Copy link
Member Author

xhub commented Mar 17, 2016

So a list of items to do before changing the license:

  • Move all the code derived from csparse to externals/SuiteSparse/CSparse/ under the name csparse_extentions. There is for sure things in numerics/src/tools/SparseMatrix.c
  • Document the license of siconos-externals being LGPLv2.1+ Make sure that this is true. ODEPACK is public domain, HEM5 looks BSD, but there is MA28 included and should be used for research only, see linsp.f. Document this. See also [cmake] add options to enforce license #33
  • do a review of the code in numerics to look for code that could come from csparse. I think that apart from the part in numerics/src/tools/SparseMatrix.c that I introduce, we are fine.

@vacary
Copy link
Member

vacary commented Mar 22, 2016

@xhub : "Move all the code derived from csparse to externals/SuiteSparse/CSparse/ under the name csparse_extentions. There is for sure things in numerics/src/tools/SparseMatrix.c"

How do you want to do it in practice. We can make a move of numerics/src/tools/SparseMatrix.c[.h] in externals/SuiteSparse/CSparse_extensions but why it will solve the problem in a proper way ?
what is the goal: to remove all the file that need a direct include of csparse.h ?

I will do that in brute force way and see how we can try to do better

@vacary
Copy link
Member

vacary commented Mar 22, 2016

I commented the commit (d6fdf1a) it is just a first start

@xhub
Copy link
Member Author

xhub commented Mar 22, 2016

@xhub : "Move all the code derived from csparse to externals/SuiteSparse/CSparse/ under the name csparse_extentions. There is for sure things in numerics/src/tools/SparseMatrix.c"

How do you want to do it in practice. We can make a move of numerics/src/tools/SparseMatrix.c[.h] in externals/SuiteSparse/CSparse_extensions but why it will solve the problem in a proper way ?
what is the goal: to remove all the file that need a direct include of csparse.h ?

The goal was to remove every piece of code that was derived from CSparse because it would infringe for sure the LGPL. I was sure that this was the case for half of SparseMatrix.

Now just using the data structure and calling the functions is fine.

By looking quickly at the code, it looks like it is the only place where CSparse source code was used.

To nitpick, I think this test: numerics/src/tools/test/SparseMatrix_test0.c should belong to the external dir, or be removed, but just because it does make sense to have it in numerics

@vacary
Copy link
Member

vacary commented Mar 23, 2016

nitpick ? Have a translation for french dummies ?
Normally, we also have to specify our modification/extension in a notice file ? I am right ?

@radarsat1
Copy link
Contributor

nitpick = pointing out very small, sometimes unimportant, details :)

@bremond
Copy link
Contributor

bremond commented Jun 11, 2016

GPL -> Apache done in 4.0 & master

@bremond bremond closed this as completed Jun 11, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants