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

Read also NOTICE file from the repository, as used/required by the Apache 2.0 license #1683

Closed
mitar opened this issue Feb 29, 2024 · 4 comments

Comments

@mitar
Copy link

mitar commented Feb 29, 2024

First, thank you for this great plugin. It really nicely solves a big paint point of dealing with licenses.

Apache 2.0 license requires to bundle also any NOTICE file, if one exists in the repository, not just the LICENSE file. Could you please read that file as well and append it to licenseText, if it exists? Currently the code reads only LICENSE* file?

@mjeanroy
Copy link
Owner

mjeanroy commented Mar 4, 2024

Hi @mitar,

Thanks for your report!

It's definitely doable, thanks I didn't know the Apache 2.0 licence required the NOTICE file to be bundled as well, would you have a link I can read?

It seems quite easy to do, the only question I have is if it should be an option, or not. Said differently:

  • I'm not sure it's desired by all users of this plugin, but it's a requirement by Apache 2.0, then it probably is.
  • I don't think adding the NOTICE would be a breaking change, but I might be wrong.

@mitar
Copy link
Author

mitar commented Mar 4, 2024

From the license:

If the Work includes a "NOTICE" text file as part of its distribution, then any Derivative Works that You distribute must include a readable copy of the attribution notices contained within such NOTICE file, excluding those notices that do not pertain to any part of the Derivative Works, in at least one of the following places: within a NOTICE text file distributed as part of the Derivative Works; within the Source form or documentation, if provided along with the Derivative Works; or, within a display generated by the Derivative Works, if and wherever such third-party notices normally appear. The contents of the NOTICE file are for informational purposes only and do not modify the License. You may add Your own attribution notices within Derivative Works that You distribute, alongside or as an addendum to the NOTICE text from the Work, provided that such additional attribution notices cannot be construed as modifying the License.

@mjeanroy
Copy link
Owner

Hi @mitar,

I just published version 3.3.0 with this feature (the notice file will be included in the third party output).
Feel free to re-open if you find an issue :)

@mitar
Copy link
Author

mitar commented Mar 10, 2024

Awesome, thank you!

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

No branches or pull requests

2 participants