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 clarification? #7

Closed
jakirkham opened this issue Aug 10, 2016 · 6 comments
Closed

License clarification? #7

jakirkham opened this issue Aug 10, 2016 · 6 comments

Comments

@jakirkham
Copy link
Contributor

The license file says MIT. However, setup.py says GPL 2. FWIW it seems astroid is the old requirement and seems to be LGPL for the version specified (unless we are talking 0.x versions). So it doesn't seem to constrain this at all. Given all of this, what is the license of requirements-detector?

@carlio
Copy link
Contributor

carlio commented Aug 10, 2016

@jakirkham I think this was actually a mistake when copying classifiers from another project while I was making it python 3 compatible: aa17279#diff-2eeaed663bd0d25b7e608891384b7298R21

It was originally supposed to say MIT (and the Licence is set to that in the call to setup. So basically it just has the wrong trove classifier. I can fix that easily enough, but the Licence is definitely supposed to be MIT.

@jakirkham
Copy link
Contributor Author

Thanks for confirming as MIT, @carlio. Leaving open for the trove classifier fix.

@jakirkham
Copy link
Contributor Author

jakirkham commented Aug 10, 2016

Should be fixed by PR ( #8 ) ( #9 ).

@jakirkham
Copy link
Contributor Author

Should also ask is the copyright correct or is this a copy-pasta issue? It attributes it to Google and the AngularJS project.

@carlio
Copy link
Contributor

carlio commented Aug 19, 2016

@jakirkham Hmm no idea how that got there, must be copypasta. I'll remove.

@jakirkham
Copy link
Contributor Author

This appears to be resolved now. Closing...

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

Successfully merging a pull request may close this issue.

2 participants