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

Missing NOTICE file for Apache license #1259

Closed
mwestphal opened this issue Nov 14, 2023 · 3 comments
Closed

Missing NOTICE file for Apache license #1259

mwestphal opened this issue Nov 14, 2023 · 3 comments
Assignees

Comments

@mwestphal
Copy link

To be complete, a Apache licensed code should not only contains the LICENSE file containing the content of the Apache 2.0 license but also a NOTICE file containing the copyright, see here for an example: https://gitlab.kitware.com/osc/apache-template

Please add a NOTICE file.

@omalyshe
Copy link
Contributor

@mwestphal The original rule to apply Apache 2.0 license https://www.apache.org/licenses/LICENSE-2.0#apply says the following: "Include a copy of the Apache License, typically in a file called LICENSE, in your work, and consider also including a NOTICE file that references the License".

I read it not as a mandatory requirement but as a recommendation.

Also, note that the copyright notice is included into each source code file, e.g. https://github.com/oneapi-src/oneTBB/blob/master/include/oneapi/tbb.h

@mwestphal
Copy link
Author

mwestphal commented Nov 15, 2023

I read it not as a mandatory requirement but as a recommendation.

Absolutely, this is correct

e.g. https://github.com/oneapi-src/oneTBB/blob/master/include/oneapi/tbb.h

While this is a valid place to look for a copyright, clarifying it in a non source file would make it clearer.
This could be in a NOTICE file, or in the README.md.

In any case, I'm fine with closing this if you think no change is needed.

@omalyshe
Copy link
Contributor

I'm closing this issue since it is not a mandatory requirement.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants