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

Clarify licensing #1213

Merged
merged 1 commit into from
Jan 6, 2021
Merged

Clarify licensing #1213

merged 1 commit into from
Jan 6, 2021

Conversation

ssbarnea
Copy link
Member

@ssbarnea ssbarnea commented Jan 6, 2021

Fixes: #1188

@ssbarnea ssbarnea requested review from gundalow and acozine January 6, 2021 10:44
@ssbarnea ssbarnea added the docs label Jan 6, 2021
@gundalow gundalow requested a review from abadger January 6, 2021 10:56
Copy link
Collaborator

@gundalow gundalow left a comment

Choose a reason for hiding this comment

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

@ssbarnea Thank you for updating this.

I'm just putting -1 till I get someone else to review this.

@gundalow
Copy link
Collaborator

gundalow commented Jan 6, 2021

@tadeboro Thank you for raising this (via #1188) how does this working sound?

@ssbarnea
Copy link
Member Author

ssbarnea commented Jan 6, 2021

@gundalow While this explanation does not have any legal value, it would worth contacting our legal team for feedback on licensing.

I am also quite curious to find about other Python packages that faced the same issue. I really doubt that dynamic languages that do not effectively embed any code from the GPL dependency are affected. In the end the string of bits, as source or package does not contain any GPL code and producing it (archiving) does not make use of any GPL code, so no lawyer would ever be able to win.

On the other hand, the reality is that you cannot make practical use of the project without also installing the GPL bits.

AFAIK, that counts as weak-dependency but it worth being documented in order to avoid confusions.

Copy link
Contributor

@acozine acozine left a comment

Choose a reason for hiding this comment

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

I've tried to make the license statement easier to read and understand. Have I retained the correct meaning?

Once we've agreed on the wording, it would be nice to add links to the licenses in the text.

README.rst Outdated Show resolved Hide resolved
README.rst Outdated Show resolved Hide resolved
README.rst Outdated Show resolved Hide resolved
@ssbarnea ssbarnea force-pushed the 0/license branch 2 times, most recently from 6cc02ad to b765fa4 Compare January 6, 2021 17:04
@tadeboro
Copy link
Contributor

tadeboro commented Jan 6, 2021

The additions do solve the main issue I had: knowing what license applies to contributions. But because IANAL, I have no idea if this is enough to satisfy the GPLv3 requirements.

@abadger
Copy link

abadger commented Jan 6, 2021

This is the clearest wording I've ever seen concerning this issue. Well done ssbarnea and acozine!

This looks like it agrees with the legal advice the lawyers have always given me about similar licensing situations. +1 from me on that count.

Copy link
Collaborator

@gundalow gundalow left a comment

Choose a reason for hiding this comment

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

Given @abadger's comment I'm changing my -1 to +1
Thank you all

@@ -88,16 +89,16 @@ console_scripts =

# core will point to ansible-core>=2.11 as soon that is released
community =
ansible>=2.10
ansible>=2.10 # GPLv3+
Copy link
Collaborator

Choose a reason for hiding this comment

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

Tracking like this is a great idea, thank you

@ssbarnea ssbarnea merged commit e4d8c35 into master Jan 6, 2021
@ssbarnea ssbarnea deleted the 0/license branch January 6, 2021 18:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ansible-lint's license (MIT) is not compatible with Ansible's license (GPLv3)
5 participants