Skip to content

add LGPL license metadata to gemspec #44

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

Merged
merged 1 commit into from
Oct 30, 2014
Merged

add LGPL license metadata to gemspec #44

merged 1 commit into from
Oct 30, 2014

Conversation

ktdreyer
Copy link
Contributor

This pull request allows RubyGems.org and other tools (such as gem2rpm) to correctly report the licenses for your gem.

As indicated in the COPYING file, a small portion of the code is likely
LGPL.
@ktdreyer ktdreyer mentioned this pull request Apr 11, 2014
@ktdreyer ktdreyer changed the title add MIT and LGPL license metadata to gemspec add LGPL license metadata to gemspec Apr 11, 2014
@ktdreyer
Copy link
Contributor Author

Mind merging this in?

@piki
Copy link
Collaborator

piki commented Oct 28, 2014

Changing the license from MIT to dual-licensed would require the approval of the authors, @tmm1 and @rtomayko.

The footnote in COPYING appears to be about crediting a potentially de minimis block of code borrowed from glibc, rather than putting the entire gem under LGPL.

@ktdreyer
Copy link
Contributor Author

Hi piki, I ran this by Fedora legal team this week, and here's their response: https://lists.fedoraproject.org/pipermail/legal/2014-October/002509.html

It seems that the LGPL tag should be added to the gemspec.

@piki
Copy link
Collaborator

piki commented Oct 30, 2014

The original authors OK'd it. Dual-license it is.

piki pushed a commit that referenced this pull request Oct 30, 2014
add LGPL license metadata to gemspec
@piki piki merged commit a986ac4 into rtomayko:master Oct 30, 2014
@ktdreyer
Copy link
Contributor Author

That's really great. Thanks for merging.

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.

2 participants