-
Notifications
You must be signed in to change notification settings - Fork 575
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
Applying Apache License to source code #143
Comments
Hi @dims -- weighing in from the CNCF / LF side, yes, we are totally fine with using a short-form SPDX license identifier rather than the lengthier "standard license header". From our view, either of the models shown at http://www.apache.org/foundation/license-faq.html#Apply-My-Software is fine to use. We regularly recommend to projects that they use the shorter-form version, as a simpler and less-error-prone way to express the license for a file. |
Thanks @swinslow could we please add this to one of the markdown files? possibly https://github.com/cncf/foundation/blob/master/allowed-third-party-license-policy.md |
Sure -- I'd suggest, let me add something to a new license-notices.md as a counterpart to the existing file copyright-notices.md. It's a related concept but I think it will be helpful to keep separate from that file. |
This is great to see and something I've been thinking as we continue to explore establishing software bills of materials in Kubernetes: kubernetes/release#1837 Thanks for opening this issue, @dims, and for the feedback, @swinslow! |
Projects like Kubernetes and Containerd are using automation and file templates to apply the license:
Kubernetes (Main repo) : https://github.com/kubernetes/kubernetes/tree/master/hack/boilerplate
Containerd : https://github.com/containerd/project/tree/master/script/validate/template
However ASF guidance here seems to indicate a shorter SPDX version is fine:
http://www.apache.org/foundation/license-faq.html#Apply-My-Software
Can we please establish that this is ok or not? and document the decision in this repository?
cc @swinslow
The text was updated successfully, but these errors were encountered: