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

Appended additional BSD 3-Clause to LICENSE #1363

Merged
merged 1 commit into from
Mar 9, 2022
Merged

Conversation

oontvoo
Copy link
Member

@oontvoo oontvoo commented Mar 9, 2022

Needed for export_header module

Needed for export_header module
@dmah42 dmah42 merged commit a25f3cd into google:main Mar 9, 2022
@dmah42
Copy link
Member

dmah42 commented Mar 9, 2022

thanks

@LebedevRI
Copy link
Collaborator

This change seems to be missing about 500 words of motivation.

@LebedevRI
Copy link
Collaborator

(I would recommend reverting this immediately)

@oontvoo
Copy link
Member Author

oontvoo commented Mar 9, 2022

This was needed because PR/1321 included code that had this license (the generate_export_header).
This change should have been done there but I guess we didn't have a git license compliant-lint check.

@LebedevRI
Copy link
Collaborator

This change should be a lot more explicit about that, it should call out that said new license is only for that single specific file,
otherwise this can be read as if the whole project is now double-licensed. This is a problem.

@dmah42
Copy link
Member

dmah42 commented Mar 10, 2022

why would it be a problem that the project as a whole is double-licensed under these licenses, which are completely compatible?

@LebedevRI
Copy link
Collaborator

why would it be a problem that the project as a whole is double-licensed under these licenses, which are completely compatible?

Please consult a lawyer.
Until then, please revert this.

@dmah42
Copy link
Member

dmah42 commented Mar 10, 2022

we have very strong guidance around use of licenses, and specifically double licensed, in third party code at Google and there is no concern with mixing 'notice' style licenses.

in general, the most restrictive license would apply when parts of the code are under separate licenses. note, this applied as soon as the BSD licensed code was added, and this PR is irrelevant as it is just documenting the case. if you want to revert anything, the inclusion of the BSD-licensed code should be reverted.

however, in this case, both licenses are equally restrictive with compatible requirements so there is no issue here.

if you have an issue relating to how you can or can't use a multiply-licensed third party package then we need to look at the use of BSD licensed code, not the documentation of that code.

@LebedevRI
Copy link
Collaborator

This change should be a lot more explicit about that, it should call out that said new license is only for that single specific file, otherwise this can be read as if the whole project is now double-licensed. This is a problem.

@oontvoo
Copy link
Member Author

oontvoo commented Mar 10, 2022

@LebedevRI I've sent PR/1366 . PTAL. Thanks!

@oontvoo
Copy link
Member Author

oontvoo commented Mar 10, 2022

from my interpretation, LebedevRI's concern is that it's not OK to retrospectively re-license code (which is what this PR sort of accidentally did). I think that's a reasonable point.

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.

3 participants