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

colamd.c has an incorrect mention of LGPL license #145

Closed
rgommers opened this issue Jul 22, 2024 · 2 comments · Fixed by #149
Closed

colamd.c has an incorrect mention of LGPL license #145

rgommers opened this issue Jul 22, 2024 · 2 comments · Fixed by #149

Comments

@rgommers
Copy link

We received a SciPy bug report about colamd.c in the vendored SuperLU code claiming that the code is under the LGPL license: scipy/scipy#21234. The problematic mention is at:

terms of the GNU LGPL, provided that the Copyright, this License,

The code isn't actually LGPL-licensed; the BSD license mention at the top of the same file is correct, as discussed in more detail in the linked SciPy issue. The LGPL mention is still problematic, since there are now two conflicting licenses mentioned in the same file, and the LGPL mention is going to trigger extra work for Linux distro packagers and anyone else who uses license checkers to carefully analyze code for license compliance.

This problem was introduced in commit 44f2c5e. The right fix I believe is to remove the offending bit of code comment that contains "LGPL". I'd be happy to open a PR for that if that helps.

@gruenich
Copy link
Contributor

I think you are right. Tim Davis' colamd is licensed under BSD-3.

Please provide a PR as this increases the changes to get this fixed.

hawkinsp added a commit to hawkinsp/superlu that referenced this issue Jul 25, 2024
v4.5.6 relicensed this file under 3-clause BSD instead of LGPL.

Fixes xiaoyeli#145
@hawkinsp
Copy link
Contributor

I think this wasn't an accident: the license was LGPL but it was changed at this commit; see COLAMD/Doc/License.txt

DrTimothyAldenDavis/SuiteSparse@75753f9#diff-63df1cd034960b4145afd34464072e6a66d38c76cc530659bc2f85d0695413f0

#149 updates to v4.5.6, which is the first commit after it was relicensed to BSD.

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

Successfully merging a pull request may close this issue.

3 participants