-
Notifications
You must be signed in to change notification settings - Fork 12
License status? #17
Comments
The license change was done by @rtfarte. Perhaps he can explain? |
Don't get me wrong, I think it would be great to have as open license as possible :) But based on my understanding, changing the license of the whole project would mean we'd have to rewrite iText parts from scratch first, as stated in #6. |
If we don't hear from @rtfarte about this, then I would gladly approve a pull-request to revert the license change. |
Cannot comment other commits, but as you forked from my original code - I do not agree with license change, because it is illegal. Althought I aprreciate the work you are doing, I cannot simply agree with illegal activities. So please change it back to LGPL/MPL. Maybe for simplification (i do not like dual licenses) it will be preferable to use LGPL, but it depends on you. |
Thanks for stating your case @kulatamicuda. I'll prepare a PR. |
Now it's all good? |
Thanks everyone for noticing and handling this issue appropriately. I'll go ahead and pull these changes into the version published on Maven Central. |
@bengolder @rtfarte @kulatamicuda |
Agreed. I just hated the licensing of the original project. I'm no lawyer, but to be safe we should revert back to the original license. Thanks, On Oct 20, 2016, 11:25 AM -0600, Juha Jokimäki notifications@github.com, wrote:
|
As a side issue: the license file currently contains text from MPL 2.0 / LGPL 2.1, but the readme talks about MPL 2.0 / LGPL 3.0. I guess the license file is probably right, but then the readme should be corrected to LGPL 2.1 for clarity's sake. |
See notes in files from original itext 4.2.0 on which this whole project is originally based:
|
Sure you can decide to upgrade the project from using LGPL 2.1+ to requiring LGPL 3.0+… but having one in the license file and another in the readme file doesn't makes much sense to me: should the user follow the 2.1 as extensively stated in the license file or the 3.0 as succinctly stated in the readme file? |
Reverse commit of b7af357 + updated readme.
I noticed that you changed the license to MIT in b7af357. However, the original iText source files state that they are licensed under LGPL/MPL. I assume that only new contributions are licensed under the MIT license? I don't think you can change the license of the whole code base unless you're the original copyright holder, especially from a restrictive license like LGPL to more permissive MIT.
Another question is what would be the license of the whole library. I'm under the impression that because LGPL is a copyleft license, the whole work would have to be licensed under that?
Perhaps you could clarify the license status in README? In my opinion the clearest and (from a legal perspective) safest thing to do would be to revert back to the original license. But again, I'm not a legal expert on FOSS licenses and I would just like to know your reasoning.
The text was updated successfully, but these errors were encountered: