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

LICENSE and NOTICE #26

Merged
merged 1 commit into from
Jun 25, 2022
Merged

LICENSE and NOTICE #26

merged 1 commit into from
Jun 25, 2022

Conversation

rossabaker
Copy link
Member

I added this to the http4s fork, and I think they're as relevant here.

  • It is common to provide a full copy of the Apache 2 license as LICENSE
  • The LICENSE doesn't actually claim copyright. That's what NOTICE is for.

NOTICE adds a year that we need to remember to bump.

@DavidGregory084
Copy link
Member

Does this mean that if there's a notice file, we don't need headers @rossabaker? I've noticed that Rust projects deliberately don't use per-file headers, see e.g. rust-lang/rust#43498 or rust-lang/cargo#4331

@armanbilge
Copy link
Member

Linking to typelevel/sbt-typelevel#72 which I still think is a good idea.

@DavidGregory084
Copy link
Member

Actually, tbf the specific objection on the rust-lang repos was that the Rust Project Contributors was not a legal entity, whereas Typelevel is

@rossabaker
Copy link
Member Author

rossabaker commented Jun 9, 2022

The ASF still requires it, but that's for ASF, not for non-ASF Apache licensed code.

The Apache license generally proposes the header "to apply the Apache License to specific files in your work". It is irritatingly ambiguous whether it must be specifically applied to all files in the work, or for exceptions to some ambient default license, which is why unqualified people like us debate this every few months. 😆

I am on Team Put It Everywhere because:

  • Corporate lawyers at past jobs have told me put it everywhere.
  • ASF, who made this license, puts it everywhere.
  • Attribution of derived work is required, and this makes it easy.

@valencik
Copy link
Member

I think the discussion landed on this being good to go, merging.

@valencik valencik merged commit c8d4e2c into typelevel:main Jun 25, 2022
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.

4 participants