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 GNU General Public License V3 license #60

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

dkorn
Copy link
Owner

@dkorn dkorn commented Mar 19, 2017

@cben
Copy link
Collaborator

cben commented Mar 20, 2017

LGTM (or should it be GTM? LGTM would be the Lesser Good To Me ;-)

I'd also add something like this to the README, to spare people opening the file (though the name COPYING is a strong hint):

# License
[GPL version 3](COPYING), same as Ansible.

@dkorn dkorn force-pushed the add_gplv3_license branch 2 times, most recently from 29c7f6b to 80ad868 Compare March 20, 2017 12:14
@dkorn
Copy link
Owner Author

dkorn commented Mar 20, 2017

I'd also add something like this to the README, to spare people opening the file (though the name COPYING is a strong hint):

Done, thanks!

@cben
Copy link
Collaborator

cben commented Mar 20, 2017

The commit message contains @cben @simon3z PTAL, might want to rewrite that :)

@dkorn
Copy link
Owner Author

dkorn commented Mar 22, 2017

The commit message contains @cben @simon3z PTAL, might want to rewrite that :)

Done, thanks :)

@cben
Copy link
Collaborator

cben commented Apr 25, 2017

All contributors, please confirm you're fine with GPLv3 (this is Ansible's own license, and what is required for eventually contributing the modules to Ansible).

Alphabitically (includes everybody in git log, could have missed a reviewer:

(please reply and I'll update this checklist)

P.S. https://docs.ansible.com/ansible/dev_guide/developing_modules_in_groups.html#before-you-start-coding says "Shared code can be placed into lib/ansible/module_utils/, should be done so under the BSD license."
Not sure if this will apply to code sharing we'll want (#36), and if we can handle it ahead of time (e.g. have dual GPL + BSD license in this repo?).

Copy link
Contributor

@enoodle enoodle left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I approve

Copy link
Contributor

@zgalor zgalor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I concur

@cben cben mentioned this pull request Apr 25, 2017
41 tasks
@theute
Copy link
Contributor

theute commented Apr 26, 2017

Sure

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.

None yet

5 participants