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

Apache-1.1 license obligations #185

Closed
waynebeaton opened this issue Sep 29, 2022 · 3 comments · Fixed by #186
Closed

Apache-1.1 license obligations #185

waynebeaton opened this issue Sep 29, 2022 · 3 comments · Fixed by #186
Assignees
Labels
documentation Improvements or additions to documentation
Milestone

Comments

@waynebeaton
Copy link

waynebeaton commented Sep 29, 2022

The project has some content that is under the Apache-1.1 license (see org.aspectj.apache/bcel/Repository.java).

According to the Apache-1.1 license:

Redistributions in binary form must reproduce the above copyright notice, this list of conditions and the following disclaimer in the documentation and/or other materials provided with the distribution.

AFAICT, this means that the license text in its entirety needs to be included in any JAR files that include Apache-1.1 content (e.g., the aspectjweaver.jar file).

Further, end user documentation needs to include a statement:

 * 3. The end-user documentation included with the redistribution,
 *    if any, must include the following acknowledgment:
 *       "This product includes software developed by the
 *        Apache Software Foundation (http://www.apache.org/)."
 *    Alternately, this acknowledgment may appear in the software itself,
 *    if and wherever such third-party acknowledgments normally appear.

I believe that including the statement in the README probably gets us there.

Can a committer please make these changes in time for your next release? I do not believe that there is any need to go back and "fix" any existing releases.

@kriegaex
Copy link
Contributor

Thanks for bringing up this issue. The AspectJ project does, however, acknowledge the use of BCEL, also mentioning its licence:

https://github.com/eclipse/org.aspectj/blob/67b1c353a02c335074a736ebf76a49bf24eefb19/docs/dist/LICENSE-AspectJ.html#L41-L46

Would that satisfy your requirement, or is there still a gap?

@kriegaex
Copy link
Contributor

kriegaex commented Sep 29, 2022

I just checked. This licence file has always been part of the AspectJ installer download until version 1.9.6 (inclusive), which you can verify by downloading and installing AspectJ from https://www.eclipse.org/aspectj/. As of 1.9.7, this is no longer true, because after some Maven-related changes, we no longer build the product in the same way. It is my own oversight that I omitted the licence file as part of the documentation, which I no longer generated into the installer package, because I think that the online documentation is just fine. I am going to make sure that the licence file is going to be part of future releases again. Would it suffice to put the licence file into the installer package, or does it need to be part of all public artifacts, such as AspectJ runtime, weaver and tools?

@kriegaex kriegaex self-assigned this Sep 29, 2022
@kriegaex kriegaex added the documentation Improvements or additions to documentation label Sep 29, 2022
@kriegaex kriegaex added this to the 1.9.10 milestone Sep 29, 2022
@waynebeaton
Copy link
Author

Thanks for your prompt attention.

I'm sorry that I missed the reference in the documentation. So that should cover that.

My interpretation of the license is that it should be in all public artifacts. Ideally, somebody obtaining a JAR file from Maven Central should get the license.

kriegaex added a commit to kriegaex/aspectj that referenced this issue Oct 1, 2022
The license file will be included right in the main directory of both
binary and source artifacts for
  - AspectJ Matcher,
  - AspectJ Runtime,
  - AspectJ Weaver,
  - AspectJ Tools (Compiler),
  - AspectJ Installer.

Fixes eclipse-aspectj#185.

Signed-off-by: Alexander Kriegisch <Alexander@Kriegisch.name>
kriegaex added a commit that referenced this issue Oct 1, 2022
The license file will be included right in the main directory of both
binary and source artifacts for
  - AspectJ Matcher,
  - AspectJ Runtime,
  - AspectJ Weaver,
  - AspectJ Tools (Compiler),
  - AspectJ Installer.

Fixes #185.

Signed-off-by: Alexander Kriegisch <Alexander@Kriegisch.name>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants