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

GPL license on fy-list.h #63

Open
Enjection opened this issue Dec 20, 2022 · 3 comments
Open

GPL license on fy-list.h #63

Enjection opened this issue Dec 20, 2022 · 3 comments

Comments

@Enjection
Copy link

It seem's that it is forbidden to licesne your library under MIT license with code licensed under GPL2.0 in it.

@pantoniou
Copy link
Owner

Up to now people didn't object much (GPL2 is much less viral than GPL3).

Another reason is that using the Linux kernel's list implementation makes things easier to integrate in the kernel (when/if that time comes).

For commercial applications a pure MIT library has merits.

I will continue this discussion at the pull request...

@bkmgit
Copy link

bkmgit commented Jul 12, 2023

Maybe it is helpful to indicate in the README that fy-list.h is GPL2 and and xxhash is BSD-2-Clause

@zlobober
Copy link

zlobober commented Nov 5, 2024

I agree with @Enjection, @bkmgit and the most commenters from PR #64 that it is not nice to keep incorrect licensing information. This could possibly poison third-party projects with permissive licenses that depend on libfyaml, making them implicitly violate GPL license rules, as they rely on you stating that your project is MIT-licensed.

IMO licensing issues should be qualified as critical for open-source projects. I suggest merging either the PR #89, switching the library license to GPL or (which is better) PR #64, getting rid of GPL code and enabling easy usage of a library in open-source projects with permissive licenses.

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

4 participants