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

Update IPR policy to introduce BSD 3-Clause license #115

Closed
wants to merge 27 commits into from

Conversation

foolip
Copy link
Member

@foolip foolip commented Apr 3, 2020

Closes #114.

policy-link-mapping.txt Outdated Show resolved Hide resolved
BSD 3-Clause License.md Outdated Show resolved Hide resolved
IPR Policy.md Outdated Show resolved Hide resolved
@foolip
Copy link
Member Author

foolip commented May 18, 2020

45 days have passed and we should now merge this and go ahead with the implementation in each spec repo. @dbaron @travisleithead, there's nothing new here, but can I have your approval, given that this is changing our policies?

@annevk
Copy link
Member

annevk commented May 19, 2020

I don't mind doing this, but I think it would be somewhat preferable if we also worked out the changes to https://github.com/whatwg/spec-factory and individual specifications.

Shouldn't https://whatwg.org/workstream-policy#notice-for-living-standards change as well?

And https://github.com/whatwg/participate.whatwg.org/blob/master/views/agreement.hbs?

@othermaciej
Copy link
Contributor

Yes, the notice should change. It would be an improvement to do that in the same PR. Suggestion:

Copyright © YEAR WHATWG (Apple, Google, Mozilla, Microsoft). This work is licensed under a Creative Commons Attribution 4.0 International License. Alternatively, this work is licensed under the [BSD 3-Clause License] for incorporation into software.

I also agree that the participant agreement should also be updated to link the new definitions in the IPR Policy. Too tired to have a concrete suggestion.

@othermaciej
Copy link
Contributor

(My suggestion should probably have at least cursory legal review.)

@foolip
Copy link
Member Author

foolip commented May 19, 2020

I've made a checklist in #114 (comment). I'll take a look at the Workstream Policy.md change now.

@foolip
Copy link
Member Author

foolip commented May 19, 2020

@othermaciej, I have applied your suggestion on notice wording in 5e32a85 and therefore also rerequested your review.

That change also updates the LICENSE file, which would then be copied verbatim to LICENSE.template in spec-factory.

To explain why these changes are only being done now, the original plan called for "For purposes of implementing a WHATWG Living Standard or Review Draft, WHATWG releases portions of the Living Standard, Review Draft or documents referenced therein under the BSD 3-Clause License for incorporation into software." to be the wording in our IPR policy, in specs and in LICENSE files.

However, the wording across policy and notices isn't the exact same for the CC-BY license, and putting "Living Standard or Review Draft" in a document which we know to be one or the other looks odd. Maciej's suggestion for the notice wording seems sensible to me.

But, I think we'll have to circle back to get legal input on that notice wording.

Workstream Policy.md Outdated Show resolved Hide resolved
foolip added a commit to whatwg/spec-factory that referenced this pull request May 19, 2020
LICENSE Outdated Show resolved Hide resolved
LICENSE Outdated Show resolved Hide resolved
Copy link
Member

@dbaron dbaron left a comment

Choose a reason for hiding this comment

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

My one concern is that the edit to IPR Policy.md says "portions of", whereas the edit to LICENSE and the 3 (?) edits to Workstream Policy.md say "this work". That seems a bit inconsistent. (Were those 2 sets of changes made at different times?)

@othermaciej
Copy link
Contributor

othermaciej commented May 19, 2020

The scope of "portions of" was unclear so I wasn't sure what to say. Does "portions of" mean portions we are supposed to identify up front for each spec, or does it mean whoever is using it may choose any portion? I was worried that saying "portions of" in the copyright notice would suggest to people something even less clear, which is that some portions are dual-licensed, but we can't say which portions because we're not sure.

Expanding a bit: I think we specifically rejected this dual license being limited to things like IDL, data files or code samples. So that's not what the "portions" language means. I'm assuming it's a portion chosen by whoever is copying from the spec, and thus perhaps equivalent to dual-licensing the whole work.

foolip added a commit to whatwg/spec-factory that referenced this pull request May 25, 2020
@foolip
Copy link
Member Author

foolip commented May 26, 2020

The SG has met with our legal counsel and asked if we can instead go with a more straightforward dual-licensing wording in the copyright notice.

@othermaciej
Copy link
Contributor

Could we consider splitting this PR? It seems there's no complications or open issues with the "field of web technologies" definition, so that could land now. But the BSD 3-clause change requires changes to additional documents and potentially legal advice, so not quite ready to land. It would be a shame if the "field of web technologies" improvement was also held up because of this.

@othermaciej
Copy link
Contributor

For the "field of web technologies" portion of this change, ideally the Contributor and Workstream Participant Agreement should be updated to link "work in the field of web technologies" to the new definition in the IPR policy. That's in a separate repo, so would have to be in a separate PR.

@foolip
Copy link
Member Author

foolip commented Jun 15, 2020

@othermaciej good idea, I've sent #129 for just the FoWT update. I'll also prepare a change for participate.whatwg.org.

@foolip
Copy link
Member Author

foolip commented Nov 6, 2020

This PR has been updated by integrating changes from #149 and some additional changes by me.

New review of all @whatwg/sg would be great.

I'd like to call special attention to 6dad672. The wording for "Other documents" was updated back in ac8dc6f but then not fixed for the new "to the extent" wording. To resolve it I've suggested simply "Documents other than [Living Standards] and [Review Drafts] are published under a license (or license waiver) specified by the [Steering Group]"

License Sample For Source Code.txt Show resolved Hide resolved
Copy link
Member

@travisleithead travisleithead left a comment

Choose a reason for hiding this comment

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

Most recent updates LGTM. Note, there is an outstanding issue about the Notice text language that Maciej raised for additional clarity that the the LWG are considering.

Copy link
Member

@travisleithead travisleithead left a comment

Choose a reason for hiding this comment

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

Feedback from the SG/LWG meeting 11/20/2020.

IPR Policy.md Outdated Show resolved Hide resolved
LICENSE Show resolved Hide resolved
LICENSE Outdated Show resolved Hide resolved
LICENSE Outdated Show resolved Hide resolved
IPR Policy.md Outdated Show resolved Hide resolved
Workstream Policy.md Outdated Show resolved Hide resolved
Workstream Policy.md Outdated Show resolved Hide resolved
Workstream Policy.md Outdated Show resolved Hide resolved
Workstream Policy.md Outdated Show resolved Hide resolved
Workstream Policy.md Outdated Show resolved Hide resolved
Co-authored-by: Travis Leithead <travil@microsoft.com>
@foolip
Copy link
Member Author

foolip commented Nov 25, 2020

I'm not able to approve as I created this PR, but I'm happy with the state of the PR after applying the changes.

All four SG members have approved at some point now, and while another look might be good from someone, I think we're good to go here.

annevk
annevk previously requested changes Dec 1, 2020
Copy link
Member

@annevk annevk left a comment

Choose a reason for hiding this comment

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

I'm pretty sure there was a request to not self-host the BSD license and link to https://opensource.org/licenses/BSD-3-Clause in the respective places instead.

@annevk annevk dismissed their stale review December 14, 2020 14:41

Addressed via commit.

Base automatically changed from master to main January 15, 2021 07:07
foolip added a commit that referenced this pull request Jan 27, 2021
@foolip
Copy link
Member Author

foolip commented Jan 27, 2021

This and #148 have been combined into #154. Minor differences have been resolved, and #154 is now exactly equivalent to merging these two older PR branches together.

@foolip foolip closed this Jan 27, 2021
@foolip foolip deleted the ipr-policy-update branch January 27, 2021 13:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

Policy Change: Field of Web Technologies definition and copyright license
8 participants