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

Add license files: GPLv2 + LGPLv2.1 #75

Merged
merged 1 commit into from
Aug 29, 2019

Conversation

filbranden
Copy link
Member

Licensing information was already present in README.md:

  • numactl and the demo programs are under the GNU General Public License, v.2.
  • libnuma is under the GNU Lesser General Public License, v2.1.

However, we weren't shipping a copy of the license text, so let's retify that by including a copy of the text of both GPLv2 and LGPLv2.1.

Also update Makefile.am to include the LICENSE.* files in a dist tarball.

Fixes #68.

Licensing information was already present in README.md:

 - `numactl` and the demo programs are under the GNU General Public License, v.2.
 - `libnuma` is under the GNU Lesser General Public License, v2.1.

However, we weren't shipping a copy of the license text, so let's retify
that by including a copy of the text of both GPLv2 and LGPLv2.1.

Also update Makefile.am to include the LICENSE.* files in a dist tarball.

Signed-off-by: Filipe Brandenburger <filbranden@gmail.com>
@filbranden
Copy link
Member Author

NOTE: I'm planning to cut a new release as soon as this commit is merged. It's been a while since we released, let's do so shortly.

@jakirkham
Copy link

Thanks for working on this, @filbranden!

Would it make sense to break out this section from the README as a license file as well?

Also noticed the manpages appear to be under The "Verbatim" License. Not sure if that should be included as a separate file too, but figured it was worth raising for discussion.

@filbranden
Copy link
Member Author

Would it make sense to break out this section from the README as a license file as well?

No, I don't think so. The section in README explains which files are covered by which license. That's where that information should be. The LICENSE.* files are simply an unmodified copy of the full license text and that's it.

Also noticed the manpages appear to be under The "Verbatim" License. Not sure if that should be included as a separate file too, but figured it was worth raising for discussion.

Thanks for pointing that out.

I wouldn't touch that now, actually. For compliance reasons, those files already include the full license text, so you can find it there.

I think we might end up simplifying that by relicensing them under the LGPL as well. (In a follow up step.)

I would like to follow up and add SPDX headers to the files to simplify compliance even further. At that point, we might want to bring this up again and possibly relicense them.

@filbranden
Copy link
Member Author

@andikleen If you don't have any strong objections, I would like to get this PR merged.

Once this one is in, I'd like to do release a new package.

Cheers!
Filipe

@filbranden
Copy link
Member Author

Friendly ping @andikleen

@andikleen
Copy link
Contributor

Yes it's ok for me. Sorry for the delay.

@filbranden filbranden merged commit b4d36f6 into numactl:master Aug 29, 2019
@filbranden filbranden deleted the license1 branch August 29, 2019 23:23
@jakirkham
Copy link

Thanks @filbranden! 😀

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 this pull request may close these issues.

Adding the license file
3 participants