-
Notifications
You must be signed in to change notification settings - Fork 39
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 of extracted rules #23
Comments
Thanks for bringing this up. During development I was mainly going off of this comment (from here):
I am not sure in how far LGPLv2.1 restricts resources used by the LGPLv2.1 licensed application, to me it seems like it is mainly intended to protect source code (from TLDRLegal):
In addition there is the fact that the relevant That said, I am by no means well versed in legal stuff so not sure how relevant this information is. @danielnaber would be great if you could chime in here :) |
I looked into this a bit just now. I believe there is no significant problem. From https://stackoverflow.com/a/10179181 and http://www.gnu.org/licenses/gpl-faq.html#LGPLStaticVsDynamic:
As I see it including the binaries at compile time would clearly fall under the second point because everything is provided to use a different binary by nlprule being Open Source. If I understand correctly the binaries have to be licensed under LGPLv2.1 since they are derived from LanguageTool. If the binaries are included at compile time, NLPRule is comparable to a library that statically links to an LGPLv2.1 library. This does not have any implications on the licensing of NLPRule itself and should thus be irrelevant to a user. There has to be a notice that the binaries are LGPLv2.1 though. |
I don't see any issue with having the rules in the binary and having it released under a different license, as long as it's clear which rules those are (as in "rules from LanguageTool 5.2", or a specific LanguageTool commit id, or having the rules in the nlprule source code). |
Great, thanks! There will be a note that the binaries are licensed under LGPLv2.1 and derived from LanguageTool vX.X then. nlprule will stay licensed under MIT OR Apache 2.0 (actually only the Rust core is alternatively under Apache 2.0 at the moment but I'll fix that). @drahnr please confirm if that adresses your concern. |
Feel free to close this as soon as the ref to the precise language tool version from which the artifacts were extracted are present. Thanks for looking into this! |
I had brief look into the licensing of language tools rules, if they are permitted to be distributed under other licenses than the language tool library itself, which is LGPLv2.1.
Mostly in relation to #12 which would render the whole idea of including it at compile time rather pointless for most applications.
The text was updated successfully, but these errors were encountered: