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 license guidelines for projects #66

Merged
merged 2 commits into from
Oct 14, 2019
Merged

Conversation

jasongrout
Copy link
Member

@jasongrout jasongrout commented Sep 9, 2019

This follows up on conversation in #37 and the mailing list discussion about updating our project license and copyright terms to be more clear and conform better to the opensource.org and spdx BSD license pages.

Fixes #37

Summary of updates

  • Update primary license name to 3-Clause BSD License to reflect the opensource.org and spdx.org primary names for the license

  • Make copyright owner “Project Jupyter Contributors” to reflect discussion in the mailing list thread. We do not view this as a change, but rather as a clarification of our shared copyright model.

  • Wrap all lines to 78 or fewer characters

  • Update BSD conditions to be enumerated with numbers, following the text on opensource.org and spdx.org

  • In condition 3, replace “Jupyter Development Team” with the exact license text “copyright holder”.

  • We previously had, in the disclaimer section, a sentence starting with “IN NO EVENT SHALL THE COPYRIGHT OWNER OR CONTRIBUTORS…”. OWNER is now replaced with HOLDER, to reflect the license text on opensource.org and spdx.org

  • Update the per-file license statement with “Project Jupyter Contributors” and “3-Clause BSD License” terminology.

I also edited our copyright policy explanation for grammar and to clarify that for a particular project, its copyright is shared by its contributors (not necessarily all Jupyter contributors).

These changes come from discussion at jupyter#37 and on the mailing list thread https://groups.google.com/forum/#!topic/jupyter/vZpWgw8zKdc

Summary of changes:

* Update primary license name to 3-Clause BSD License to reflect the opensource.org and spdx.org primary names for the license

* Make copyright owner “Project Jupyter Contributors” to reflect discussion in the mailing list thread. We do not view this as a change, but rather as a clarification of our shared copyright model.

* Wrap all columns to 78 or fewer characters

* Update BSD conditions to be enumerated with numbers, following the text on opensource.org and spdx.org

* In condition 3, replace “Jupyter Development Team” with the exact license text “copyright holder”.

* We previously had, in the disclaimer section, a sentence starting with “IN NO EVENT SHALL THE COPYRIGHT OWNER OR CONTRIBUTORS…”. OWNER is now replaced with HOLDER, to reflect the license text on opensource.org and spdx.org

* Update the per-file license statement with “Project Jupyter Contributors” and “3-Clause BSD License” terminology.
This also clarifies that for a particular project, its copyright is shared by its contributors (not necessarily *all* Jupyter contributors).
@jasongrout
Copy link
Member Author

According to governance rules, this PR needs approval by 2/3 of the steering council, with at least 80% voting, to be merged. For our current size of 17, that means at least 14 total votes and at least 12 positive votes. Count my vote as 1.

Of course community discussion is welcome as well.

@jasongrout
Copy link
Member Author

And of course, comments or suggestions are also encouraged and invited, not just an up-or-down vote.

Instead, Project Jupyter source code is the collective copyright of its
contributors. If individual contributors want to maintain a record of what
changes or contributions they have specific copyright on, they should indicate
their copyright in the commit message of the change when they commit the
Copy link
Member

@afshin afshin Sep 9, 2019

Choose a reason for hiding this comment

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

I don't have an objection to this language because it specifically says this is to maintain a record. It still remains the case that a copyright holder's rights are intact irrespective of whether they explicitly say so.

The language in this clause does make it sound a little bit like those copyright notices should be in commit messages, but I think a close reading of this is actually correct, so I am not proposing a change. I am commenting here for clarity and also to invite disagreement if I've misunderstood.

Copy link
Member Author

Choose a reason for hiding this comment

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

I had the same thoughts - the existing language was technically fine (though a bit awkward grammatically?), so I hesitated to change it too much.

Copy link
Member

@fperez fperez left a comment

Choose a reason for hiding this comment

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

These look good to me, no objection. Thanks @jasongrout!


Copyright (c) YEAR Project Jupyter Contributors.
Copy link
Member

Choose a reason for hiding this comment

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

should YEAR here remain a placeholder, or do we put 2015, cribbing from the proposed deletion?

Copy link
Member Author

Choose a reason for hiding this comment

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

Good question. I figured the year would change depending on when that specific repo was started, so I left it a placeholder.

(also known as New or Revised or 3-Clause BSD), as follows:

- Copyright (c) 2001-2015, IPython Development Team
- Copyright (c) 2015-, Jupyter Development Team
Copy link
Member

Choose a reason for hiding this comment

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

does this deletion retroactively change the copyright that used to be there? I would have thought that any previous BSD license copyrights would remain in place, but maybe it's not a big deal here.

Copy link
Member Author

@jasongrout jasongrout Sep 11, 2019

Choose a reason for hiding this comment

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

I think probably Jupyter Development Team can be (optionally?) changed to Project Jupyter Contributors. I think any other copyrights probably ought to be left in place, and the Project Jupyter Contributors copyright should be appended. Thoughts?

Copy link
Member

Choose a reason for hiding this comment

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

I think any other copyrights probably ought to be left in place, and the Project Jupyter Contributors copyright should be appended.

Makes sense to me.

@ivanov
Copy link
Member

ivanov commented Sep 11, 2019

cookies for anyone coming here from the Steering Council email list. 🍪

@ivanov
Copy link
Member

ivanov commented Sep 11, 2019

🍪 🍪 🥛 🍰 🍫 🎁 ☕️

@jasongrout
Copy link
Member Author

Update: we've had 9 votes, all positive (as well as a bit of discussion). If you haven't voted, please vote or make a suggestion. I notice that the github approval mechanism does not have a way to vote negative, but we can count the "request changes" option as a negative vote.

@Ruv7
Copy link
Contributor

Ruv7 commented Sep 25, 2019

@jasongrout - I will abstain on this vote (I don't know enough about this topic).

@jasongrout
Copy link
Member Author

Good point, I hadn't considered abstaining. It seems reasonable that abstaining adjusts the total universe of voting members, so the threshold statement above becomes:

According to governance rules, this PR needs approval by 2/3 of the steering council, with at least 80% voting, to be merged. For our current size of 17 minus one abstaining, that means at least 13 total votes and at least 11 positive votes.

@ellisonbg
Copy link
Contributor

ellisonbg commented Sep 26, 2019 via email

@vidartf
Copy link

vidartf commented Sep 26, 2019

According to governance rules, this PR needs approval by 2/3 of the steering council, with at least 80% voting, to be merged. For our current size of 17 minus one abstaining, that means at least 13 total votes and at least 11 positive votes.

Note that you should probably define a deadline for voting, unless this is a first-past-the-post system, as a winning condition (11 of 13 votes) can be made losing (11 of 17 votes) by late voters.

@jasongrout
Copy link
Member Author

Note that you should probably define a deadline for voting, unless this is a first-past-the-post system, as a winning condition (11 of 13 votes) can be made losing (11 of 17 votes) by late voters.

Good point. I'm going by the book here (i.e., the governance docs have an explicit voting threshold for governance repo changes), which does not have a deadline mechanism for votes.

@jasongrout
Copy link
Member Author

Good point. I'm going by the book here (i.e., the governance docs have an explicit voting threshold for governance repo changes), which does not have a deadline mechanism for votes.

On the other hand, to address your point more directly - perhaps after it passes the quorum threshold of 80%, a reasonable thing to do is to then set a timeframe for any remaining votes before calling the decision.

Copy link
Member

@rgbkrk rgbkrk left a comment

Choose a reason for hiding this comment

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

This fits our spirit in embracing the copyright owners as the "Project Jupyter Contributors". Thank you @jasongrout for pushing this through.

@jasongrout
Copy link
Member Author

Thank you to everyone that responded. We now have quorum (13), with all positive votes, which means that this PR is approved. I'll leave it open until Monday for any remaining dissenting votes or more discussion, then merge Monday if there are no objections.

Thanks again everyone!

@Carreau
Copy link
Member

Carreau commented Oct 13, 2019

Thanks Jason for the ping.

Approved.

@rgbkrk rgbkrk merged commit d91f344 into jupyter:master Oct 14, 2019
@rgbkrk
Copy link
Member

rgbkrk commented Oct 14, 2019

Quorum reached, merging.

@jasongrout
Copy link
Member Author

Thanks Kyle!

@jasongrout
Copy link
Member Author

And thanks again everyone else for participating and adding your thoughts!

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.

LICENCEs Guideline