-
-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Revert "Close #628. Switch to text-unidecode" #990
Conversation
This reverts commit e32a0cd.
Hi @jorti we used to use Could you provide any documentation to your statement the the artistic license is not FOSS compatible? I see the artistic license listed as approved on opensource.org |
I'm the Fedora packager of faker and the artistic license is forbidden by the Fedora policy: https://fedoraproject.org/wiki/Licensing:Main#Bad_Licenses More info: http://www.gnu.org/licenses/license-list.html#ArtisticLicense |
If I'm understanding this correctly, Ultimately, the issue is that we don't want Looks like |
45b5ac2
to
de6af16
Compare
@fcurella Yes, looks like they clarified the license which is now Artistic 1.0 or GPL 2+. So now it would fail both the requirements stated above. It is now GPL and not FSF approved. |
@tbrlpld Perl license (dual Artistic and GPL) should be fine for Fedora: https://fedoraproject.org/wiki/Licensing:Main#Good_Licenses, this is exactly a license text-unidecode is using. |
@kmike How does one decide which of the two licenses applies? There seems to be different understanding between Perl and GNU if linking to the library (i.e. import in Python) creates the requirement to make the linking application/library also GPL.
-- https://dev.perl.org/licenses/ This is similar to what the Artistic License is stating.
-- https://dev.perl.org/licenses/artistic.html But, this seems to be conflicting with how the FSF interprets GPL:
So if I an application/library links to |
@jorti Can you confirm if GPLv2 + Artistic V1 works for Fedora? |
These are good questions @tbrlpld. My feeling is that the licenses may work out ok. Fedora prohibits packaging Artistic 1.0-only packages as they're not considered free - but Perl license it not prohibited, as users of the package can use GPL terms if they want. python-faker also can use text-unidecode, even in Fedora, as using an Artistic-licensed or Perl-licensed package doesn't put restrictions on python-faker, according to the license terms. I don't see a conflict here. Having text-unidecode in Fedora doesn't change text-unidecode license to GPL, the license is still a dual license, even if text-unidecode is executed under Fedora. There must be zillions of Perl modules packaged for Fedora, with a different mixes of licenses, there could be similar cases to look at. Sorry, I won't be able to participate in deeper license discussions, it is such a rabbit hole, and I have neither knowledge nor time to get this knowledge :) And anyways, I can't change text-unidecode license, because the license is inherited, it was not my decision to use it. |
@kmike Haha, yea I know what you are talking about. It is a rabbit hole, and I have fallen in I am afraid... Regarding you being forced to keep the Perl dual license model, this article would suggest otherwise:
The fact that your dependency it is dual licensed with GPL OR Artistic should mean that you as the "user" can choose to use the dependency under the Artistic license which does not impose the limitation to reuse the same license.
But I guess the same would then count for Faker since you are licensing with GPL OR Artistic as well would give Faker the freedom to use Sorry for ranting. I guess I am more talking to myself than anything. Just to clarify for myself: if I depend on Faker (which pulls in |
The only thing that really matters for the purpose of this issue is if the original author (@jorti ) is OK with the new license. |
Yes, the new dual licensing of text-unidecode allows us to choose GPLv2+. This is the same case as Perl, so there's no problem now. Thank you. |
Thank you @jorti ! That's all we need! :) |
This reverts commit e32a0cd.
What does this changes
Reverts the the use of text-unidecode to unidecode
What was wrong
text-unidecode is under Artistic license which is not FOSS compatible
How this fixes it
This change reverts the commit that changes to text-unidecode
Fixes #727