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

Please add a license to this repository #31

Open
arfon opened this issue Aug 12, 2015 · 5 comments
Open

Please add a license to this repository #31

arfon opened this issue Aug 12, 2015 · 5 comments

Comments

@arfon
Copy link

arfon commented Aug 12, 2015

Hi there, I'd love to use this gem but the lack of a license means I'm not able to. Would you mind adding a license to this repository? http://choosealicense.com has a good overview of some possible options.

Thanks! 😄

@SergioFierens
Copy link
Owner

That's a great point.
I have added a reference to MPL in most of the files, but no repository level license, which makes more sense.
Since I need to add it, I guess it is a good time to review if MPL is the appropriate choice. I was thinking in MIT or Apache as alternatives.
Do you have a preference? Could you share your thoughts on this?

Cheers,
Sergio

@arfon
Copy link
Author

arfon commented Aug 12, 2015

I have added a reference to MPL in most of the files, but no repository level license, which makes more sense.

Sorry I missed this 😄

I was thinking in MIT or Apache as alternatives.

I'm not a lawyer so definitely can't give you advice but I personally release all of my stuff under MIT because it's simple and permissive which is what I want. I know some people prefer Apache 2.0 because of the way it deals with patents.

In short, I think either would be great. Thanks for the response!

@frodrigo
Copy link

+1
Please, add a license file in the repository.

danbernier pushed a commit to danbernier/ai4r that referenced this issue Jun 6, 2016
Contents downloaded from
https://www.mozilla.org/media/MPL/1.1/index.0c5913925d40.txt, just a few
minutes ago.

I used MPL 1.1, per @SergioFierens' response to SergioFierens#22, where he said
MPL1.1 is the licence for this repo. Since then, the latest MPL is 2.0,
but we can handle updating the licence in another thread.

Fixes SergioFierens#31
@danbernier
Copy link

Whoops! Too fast on the PR, I missed the discussion about updating the license.

I guess you can consider this a +1 for this issue. 😃

Apologies for the noise! Before I close my PR, has there been any further thought to which license?

@bararchy
Copy link

bararchy commented Jan 9, 2017

+1 for MIT

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

5 participants