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 information to lhm.gemspec #112
Conversation
Set the license to New BSD
Out of interest, why not use the MIT license? That seems more popular amongst open source gems. |
@lgierth ? |
I'm fine with whatever license the active contributors choose 👍 |
I initially misunderstood what the ticket was about. I'm not sure about the reasons for the license choice, that was before my involvement. Unless there is a good reason, it shouldn't be changed. Nevertheless, if there are good reasons, I'm fine with whatever the active contributors decide. |
@lgierth Do you know who should I ping about license in here. Probably someone from soundcloud? |
Yeah @grobie is probably a good candidate. |
I didn't add the license and can't really comment on this. I fear a change of the license would need to be checked with legal first. Unless there are good reasons to change it let's just keep the current one for now? |
I dont think we should change.. I am just not sure if the current license is "New BSD" or not?! |
@@ -8,6 +8,7 @@ require 'lhm/version' | |||
Gem::Specification.new do |s| | |||
s.name = 'lhm' | |||
s.version = Lhm::VERSION | |||
s.licenses = ['New BSD'] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's BSD-3 I think: http://opensource.org/licenses/BSD-3-Clause
I used the License.new(
short_name: "NewBSD",
pretty_name: "New BSD",
other_names: ["Modified BSD", "BSD3", "BSD-3", "3-clause BSD", "BSD-3-Clause"],
url: "http://opensource.org/licenses/BSD-3-Clause",
matcher: matcher
)
end |
Oh, thanks, I missed that. I'm fine with any of them, something including 👍 thanks! |
'BSD-3-Clause' is less ambiguous than 'New BSD' and it's also the url slug used by the opensource.org URL http://opensource.org/licenses/BSD-3-Clause
I've updated the license name to |
Add license information to lhm.gemspec
looks good!! thanks! |
Set the license to New BSD since the current LICENSE file looks like New BSD.
This allows tools like License Finder to look up this project's metadata.
This also fixes the license display at https://rubygems.org/gems/lhm since it currently says 'N/A':