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

Missing GPL v2 + CPE license #1544

Closed
kswartz26 opened this issue Jan 7, 2022 · 7 comments · Fixed by #1545
Closed

Missing GPL v2 + CPE license #1544

kswartz26 opened this issue Jan 7, 2022 · 7 comments · Fixed by #1545
Milestone

Comments

@kswartz26
Copy link
Contributor

Our company has been using this code for a while, and we recently noticed that it contains some code under GPL v2 with Classpath Exception (https://github.com/remkop/picocli/blob/master/docs/apidocs/search.js), but we can't find a copy of that license accompanying the project. Can you please add one, either as a separate file or in a NOTICE file?

Thanks!

@remkop
Copy link
Owner

remkop commented Jan 8, 2022

Hi @kswartz26 thank you for raising this.

Will you be able to provide a pull request for this?

@kswartz26
Copy link
Contributor Author

Hopefully that will do it.

@remkop remkop linked a pull request Jan 8, 2022 that will close this issue
@remkop remkop added this to the 4.6.3 milestone Jan 8, 2022
@remkop
Copy link
Owner

remkop commented Jan 8, 2022

Thank you for the contribution! Much appreciated.

@remkop remkop closed this as completed Jan 8, 2022
@harbulot
Copy link

I can't speak as a legal expert, but using the GPL in an Apache License NOTICE file seems problematic.

It seems that GPL portion is only affecting the Javadoc output of the project, not so much the library source/binary itself. I'd generally expect the LICENSE/NOTICE files to mainly relate to the actual library code (and it's also quite common for projects to have a separate license for the documentation anyway).

It might be worth separating that, and having somewhere else than LICENSE/NOTICE instead, to make clear that the library itself is not under GPL (assuming the rest of the code doesn't have GPL inclusions, I haven't fully checked).

@remkop
Copy link
Owner

remkop commented Jul 14, 2022

@harbulot Yes that makes sense.

I will probably rename the NOTICE file to NOTICE-DOCUMENTATION, and clarify that the file in question that is under the GPL is docs/apidocs/search.js.

I will also clarify that this file is not in any binary distribution and is only included in source code distributions that include the documentation.

@harbulot
Copy link

@remkop Great, thank you.

@remkop
Copy link
Owner

remkop commented Jul 25, 2022

@harbulot I created #1750 for this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants