-
-
Notifications
You must be signed in to change notification settings - Fork 331
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
Add NOTICE file with copyright information #214
Add NOTICE file with copyright information #214
Comments
the NOTICE file is not mandatory with Apache license |
But copyright information must be available. According to my understanding even in every source file. |
I guess both of you are right. As I understood Tobias he asks you to add a file into the jar file containing a copyright notice (and not a NOTICE file). The name of the file can also be copyright.txt or something else. It would also be ok to add it into the pom.xml as comment. In my opinion Tobias is also right with the previous comment. In my opion the Apache license asks the developers to put the block
as a header (comment) into each source code file. Looking into a sample source code file (e. g. JacksonAnnotation.java) shows that the header (comment) is not there. |
I can add NOTICE in jar similar to But I definitely do not want to add tons of boilerplate in source files, that is pure unnecessary fluff. |
Actually, existing |
@cowtowncoder I'm not a lawyer but I guess the apache license is not applied as intended by apache.
|
@robertschubert you are asking for a lot of work - would you be willing to do it? There is not just this repo, there are dozens of similar repos in https://github.com/FasterXML. In the end of the day, there are arguments in favour of adding these headers to thousands of files across all these repos but of the thousands of users out there, not many are concerned. Most users are using jackson via the jars published to maven and the pom and the jars themselves fully assert the use of the Apache license. |
I do not plan on adding this boilerplate, nor would accept a PR here. |
@pjfanning Would not be a problem for me to do a contribution (with a tiny automation). But obviously it is not wanted by @cowtowncoder . Fine with that. @cowtowncoder Have a great day. |
@cowtowncoder |
@robertschubert did you not read the answer from Feb 8? |
@robertschubert That sounds reasonable. Would #219 work? |
@cowtowncoder: Thanks! I like it :) ! Fits perfectly for me! |
and: thanks for being that open for the request even if the discussion was closed already! |
@robertschubert Yes, I would want to do it over time. Although if you had time and could help, I'd be happy to merge PRs to do that. But I will start with 3 core components at least. Also: thank you for the suggestion -- I think it makes sense and hopefully helps. |
@cowtowncoder |
There's quite a few more repos with NOTICE (I think I added it to all jackson ones a month or so ago). But we can start with those; it's been 15 years until this was added so there's probably not super urgency here. |
I have created PullRequests for the modules @robertschubert mentioned: |
Merged, thank you @tobi5775 |
thanks @tobi5775 and @cowtowncoder |
For a customer project an OpenSource report has to be generated. Therefore a scanner was created to extract the copyright and licence information from our dependencies. During the process, we found that no copyright information was available for this project.
Can you please add a NOTICE file which includes the copyright notice (including your details as copyright owner) and license claim from the Appendix of the Apache 2.0 License? This will make the use of the license valid and help to get applications using this module through legal review.
Your help here would be much appreciated.
From https://www.apache.org/licenses/LICENSE-2.0:
The text was updated successfully, but these errors were encountered: