-
Notifications
You must be signed in to change notification settings - Fork 152
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
GPLv2 license #229
Comments
It looks like |
We're not planning on using the web ui in a wrapper for ios, we'll be building native apps for those with react native. We'll have to rethink the licensing for those tho. |
I missed that. I only clicked the "View license" button 😄 That particular lib should be fine then. I do, however, still think that GPLv2 might be limiting you in the long run. I'm not familiar with the Rust lib landscape though, but in Jellyfin we do use a bunch of Apache 2.0 libs. |
What license do you recommend? |
I'm not too familiar with licenses, I license my personal projects under MIT. I am only speaking from experience with GPLv2. As mentioned, GPLv3 is compatible with MIT, BSD and Apache 2.0 (and more), but not everyone likes GPLv3. It depends on what you want to achieve with the license. Maybe https://choosealicense.com/ can guide you a bit? Keep in mind that if you do re-license, you will have to ask permission from contributors. |
Noted. Thanks for the tip! |
I did a second pass and |
I can see that you have chosen the GPLv2 license, but that you're also pulling in an Apache 2.0 licensed lib (https://github.com/launchbadge/sqlx) (EDIT: Dual licensed, so ignore).
Unfortunately, to my limited knowledge (and research), those licenses are not compatible and you may, in fact, not distribute the final product under the GPLv2 license.
Have you considered other licenses? GPLv3 is compatible with Apache, but may also be more restrictive than GPLv2.
Another thing is that your web UI seems to be licensed under GPLv2 as well and if you ever plan on using the web UI in a web wrapper for iOS or other stores (smart TVs etc.), you may run into issues. The iOS app store is for instance not compatible with GPL at all.
The text was updated successfully, but these errors were encountered: