Skip to content

Make LICENSE more apparent #477

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

akirataguchi115
Copy link

Remove LICENSE.md as well due to it only referring to CONTRIBUTION. Do comment here if the license used here is wrong. I merely inferred it from the previous license text.

Copy link
Collaborator

@gregsdennis gregsdennis 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 the existing license is intentional.

@akirataguchi115
Copy link
Author

You think so? 'All rights reserved'? Could it be possible to modify it so that GitHub repository page can also show it as something it recognizes?

@gregsdennis
Copy link
Collaborator

Yes. This is an IETF project, and they likely want their license.

@cabo @timbray would you care to comment?

@cabo
Copy link
Member

cabo commented Jun 9, 2023

Ideally, the shape of the repository would be as described in RFC 8875; see https://www.rfc-editor.org/rfc/rfc8875.html#section-3.1 for one of the places this is discussed there.
This RFC was published in August 2020.
Glyn created the LICENSE file in July 2020.
So maybe we need to check whether everything is as discussed in RFC 8874/75.

@cabo
Copy link
Member

cabo commented Jun 9, 2023

'All rights reserved'?

This is a standard phrase in a copyright statement that is needed to make the copyright statement valid in certain jurisdictions.

Copyright is generally hard, because it needs to work in all jurisdictions, so I'd rather stick with what is normal here.
I'd point to an example repo by one of the people in the know, except I can't find one right now...

glyn added a commit that referenced this pull request Jun 9, 2023
extracted from this document must include Simplified BSD License text
as described in Section 4.e of the Trust Legal Provisions and are
provided without warranty as described in the Simplified BSD License.
BSD 2-Clause License
Copy link
Collaborator

Choose a reason for hiding this comment

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

The BSD license is only applicable to code components and not to specs. See https://trustee.ietf.org/
license-info.

Copy link
Author

@akirataguchi115 akirataguchi115 Jun 9, 2023

Choose a reason for hiding this comment

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

I was hoping someone would notice this. Does it become too complicated if we restructure the code components to their own sub-directory and specs to their own so that we could get two LICENSE files pointing which materials have their corresponding licenses: https://softwareengineering.stackexchange.com/questions/304874/declaring-multiple-licences-in-a-github-project#answer-304895 ?

Copy link
Member

@cabo cabo Jun 9, 2023

Choose a reason for hiding this comment

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

Does it become too complicated

I would probably answer this affirmatively.

Right now, we don't identify anything in the draft explicitly as a "code component".

The directory scripts could of course have its own license file -- none of this goes into the document, so it is useful to be explicit about the license (ignoring whether the "threshold of originality" is even reached).

The other potential code component is the ABNF, for which the scripts directory provides ways to extract it from the document, but the repo doesn't actually store the (computed) result. So there is no directory a license could be attached to.

Copy link
Author

Choose a reason for hiding this comment

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

Good reasoning. However isn't the license currently too vague and interpretable?

@timbray
Copy link
Contributor

timbray commented Jun 9, 2023 via email

@cabo
Copy link
Member

cabo commented Jun 9, 2023 via email

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.

5 participants