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

License incorrectly marked in setup.py #4382

Closed
jamalex opened this issue Sep 4, 2015 · 6 comments
Closed

License incorrectly marked in setup.py #4382

jamalex opened this issue Sep 4, 2015 · 6 comments
Assignees

Comments

@jamalex
Copy link
Member

jamalex commented Sep 4, 2015

Setup.py indicates that KA Lite is GPL, whereas it is actually MIT license:
https://github.com/learningequality/ka-lite/blob/0.15.x/setup.py#L355

@jamalex jamalex added this to the 0.14.x bugfixes milestone Sep 4, 2015
@benjaoming
Copy link
Contributor

I'm pretty sure we've discussed this already.

Maybe it has to do with the fact that we are bundling other GPL licensed libraries.

@benjaoming
Copy link
Contributor

@benjaoming
Copy link
Contributor

Possible values are listed here: https://pypi.python.org/pypi?%3Aaction=list_classifiers

Unless you wanna go through every single license in python-packages, I'd suggest accepting this "safe side" value and instead focus our efforts on cleaning up the dependency burden.

@jamalex
Copy link
Member Author

jamalex commented Sep 6, 2015

Oh, so I did actually go through python-packages, and the last time I did so, we didn't have any GPL libraries:
https://github.com/learningequality/ka-lite/blob/master/LICENSE#L38

Do you know of any that have since been added?

On the JS side, it looks like the libraries that mention GPL are "Dual licensed under the MIT or GPL":
https://github.com/learningequality/ka-lite/search?utf8=%E2%9C%93&q=gpl

If we simply depend on GPL libraries, my understanding is that doesn't force us to be GPL. However, our new "depending but then also static bundling for distribution" approach may still require GPL compliance (assuming we do actually have GPL code somewhere)?

@benjaoming
Copy link
Contributor

There are ~32 packages/modules in python-packages/, the license file lists 15 of those. So yes, more have been added for sure. I'm afraid you're the only one who took care to fill in the LICENSE file.

Anyways, searching doesn't reveal a lot of problems. Apparently requests ships with chardet bundled, which is LGPL.. but that doesn't affect our MIT license, as long as we do not remove the license description of theirs.

➜  ka-lite git:(coverage) grep -r "General Public License" python-packages
python-packages/requests/packages/chardet/gb2312prober.py:# Lesser General Public License for more details.
python-packages/requests/packages/chardet/chardistribution.py:# Lesser General Public License for more details.
python-packages/requests/packages/chardet/sbcsgroupprober.py:# Lesser General Public License for more details.
python-packages/requests/packages/chardet/langgreekmodel.py:# Lesser General Public License for more details.
python-packages/requests/packages/chardet/gb2312freq.py:# Lesser General Public License for more details.
python-packages/requests/packages/chardet/mbcsgroupprober.py:# Lesser General Public License for more details.
python-packages/requests/packages/chardet/universaldetector.py:# Lesser General Public License for more details.
python-packages/requests/packages/chardet/euctwfreq.py:# Lesser General Public License for more details.
python-packages/requests/packages/chardet/constants.py:# Lesser General Public License for more details.
python-packages/requests/packages/chardet/langhungarianmodel.py:# Lesser General Public License for more details.
python-packages/requests/packages/chardet/charsetgroupprober.py:# Lesser General Public License for more details.
python-packages/requests/packages/chardet/langcyrillicmodel.py:# Lesser General Public License for more details.
python-packages/requests/packages/chardet/escsm.py:# Lesser General Public License for more details.
python-packages/requests/packages/chardet/sjisprober.py:# Lesser General Public License for more details.
python-packages/requests/packages/chardet/codingstatemachine.py:# Lesser General Public License for more details.
python-packages/requests/packages/chardet/charsetprober.py:# Lesser General Public License for more details.
python-packages/requests/packages/chardet/euckrfreq.py:# Lesser General Public License for more details.
python-packages/requests/packages/chardet/big5prober.py:# Lesser General Public License for more details.
python-packages/requests/packages/chardet/euctwprober.py:# Lesser General Public License for more details.
python-packages/requests/packages/chardet/eucjpprober.py:# Lesser General Public License for more details.
python-packages/requests/packages/chardet/mbcharsetprober.py:# Lesser General Public License for more details.
python-packages/requests/packages/chardet/langhebrewmodel.py:# Lesser General Public License for more details.
python-packages/requests/packages/chardet/escprober.py:# Lesser General Public License for more details.
python-packages/requests/packages/chardet/utf8prober.py:# Lesser General Public License for more details.
python-packages/requests/packages/chardet/jisfreq.py:# Lesser General Public License for more details.
python-packages/requests/packages/chardet/euckrprober.py:# Lesser General Public License for more details.
python-packages/requests/packages/chardet/langthaimodel.py:# Lesser General Public License for more details.
python-packages/requests/packages/chardet/__init__.py:# Lesser General Public License for more details.
python-packages/requests/packages/chardet/big5freq.py:# Lesser General Public License for more details.
python-packages/requests/packages/chardet/latin1prober.py:# Lesser General Public License for more details.
python-packages/requests/packages/chardet/jpcntx.py:# Lesser General Public License for more details.
python-packages/requests/packages/chardet/langbulgarianmodel.py:# Lesser General Public License for more details.
python-packages/requests/packages/chardet/mbcssm.py:# Lesser General Public License for more details.
python-packages/requests/packages/chardet/sbcharsetprober.py:# Lesser General Public License for more details.
python-packages/requests/packages/chardet/hebrewprober.py:# Lesser General Public License for more details.
python-packages/requests/cacert.pem:# either the GNU General Public License Version 2 or later (the "GPL"), or
python-packages/requests/cacert.pem:# the GNU Lesser General Public License Version 2.1 or later (the "LGPL"),
python-packages/django/utils/baseconv.py:# This Baseconv distribution contains no GNU General Public Licensed (GPLed)
python-packages/httplib2/cacerts.txt:# either the GNU General Public License Version 2 or later (the "GPL"), or
python-packages/httplib2/cacerts.txt:# the GNU Lesser General Public License Version 2.1 or later (the "LGPL"),

benjaoming added a commit to benjaoming/ka-lite that referenced this issue Sep 7, 2015
jamalex added a commit that referenced this issue Sep 7, 2015
@jamalex
Copy link
Member Author

jamalex commented Sep 7, 2015

Fixed in #4384.

@jamalex jamalex closed this as completed Sep 7, 2015
@jamalex jamalex removed the has PR label Sep 7, 2015
benjaoming added a commit to benjaoming/ka-lite that referenced this issue Sep 8, 2015
benjaoming added a commit that referenced this issue Sep 8, 2015
benjaoming added a commit to benjaoming/ka-lite that referenced this issue Sep 8, 2015
benjaoming added a commit that referenced this issue Sep 8, 2015
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

No branches or pull requests

2 participants