-
Notifications
You must be signed in to change notification settings - Fork 38
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
Adding a license #94
Comments
I have no objection. |
I hadn't considered that, seems like there are conflicting opinions on this issue online.
Since UI scripts are typically single files and it's a large burden otherwise, that'd be an issue. The obvious solution is explicitly licensing the client examples under CC0. |
MPL applies to only the files that are under MPL. So for example, I can even bundle an MPL file with my proprietary software:
In this case, only modification to |
Why other scripts using thumbfast's "API" need to be GPL-ed? |
See: https://www.gnu.org/licenses/gpl-faq.html#IfLibraryIsGPL |
Is thumbfast a library? |
It certainly is in the way we traditionally use that word. I.e it provides an API for actual application (UI scripts in this case) to do things as opposed to being an end user application by itself. The gpl faq talks about dynamic/static linking, which isn't relavant for lua. But AFAIKU the concept still applies. But of course, I'm not a lawyer, so if thumbfast switches to GPL, I'll just err on the safer side and switch But if it uses MPL, then I won't have to worry about it since I do not modify the |
I think po5 should have the final say on how it is positioned and obviously it must be clear. The api it provides should not 'pollute' other scripts (or even those frontends which may use thumbfast) if possible.
|
If thumbfast is a library and uses the GPL, the scripts that use it don't follow the GPL, because they don't bundle and distribute it, thumbfast is installed manually by the user as an optional feature, and the way other scripts use thumbfast can't be called For "Link" or API. If you write a Bash Shell script and run a program that uses the GPL, is your Bash Shell script also GPL? lol... |
Why GPL3+ to begin with? |
It already happens on compiled languages. Aegisub source code is BSD but it links against GPL libraries, therefore the runtime binary is GPL. |
Linking something and calling an executable are very different. As long as other scripts only depend on thumbfast and not ship with it, their license wont be affected by the license of thumbfast afaik. But you know, not a lawyer and all that... I'm fine with making the license GPLv3. |
@AndersonTorres Other scripts do not depend on thumbfast, because it just accepts messages, and the provider of this message can be any script. |
Yup, see jellyfin-mpv-shim and marzzzello/mpv_thumbnail_script#40 this is why the API usage example should just be CC0. |
Did discussion end on this? Just ran into issues installing thumbfast on NixOS because it needs you to allow "unfree" packages (this includes packages without a license), not a blocker, just annoying, but it seems to be a problem with MPV scripts in general (a lot lack licenses). |
I'm somewhat worried that you're going to get yourself in legal trouble if your playing so loose with license stuff.
I'm fine with MPL-2.0, and I'm also fine with you using my code (I would appreciate asking though, particularly when the licenses are not compatible). I'm just worried that you might end up stepping on the wrong persons foot at some point, that's all. |
If someone wants to ruin the fun, I'm sure there will be more people mad at that person than me. None of us are getting paid and do it for the love of it. No lawyers are involved, or we wouldn't get anything done. |
Like I said I'm fine with you using it, so I won't open an issue about it and there is no need to change anything because of me. BTW I never thought I had any use for a "recently played" script, but it turned out to actually be useful 😄. Thanks for your work (not just memo but also other projects). |
I haven't had anyone request this, but it makes sense to have one.
Let's do it properly and get everyone's approval for GPLv3. 👍
@christoph-heinrich @natural-harmonia-gropius @qwerty12 @hooke007 @leiserfg @Hill-98 @dyphire
The text was updated successfully, but these errors were encountered: