-
Notifications
You must be signed in to change notification settings - Fork 128
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
Class not found on mvn license:aggregate-third-party-report #73
Comments
This also happens when executing mvn license:third-party-report. The workaround about adding doxia-core as plugin dependency worked for me, too. |
I have the same issue, and the workaround "fixes" the problem. |
Hi guys, Which version of maven-site-plugin ae you using ? |
Hi @tchemit I am getting this error with version 1.13:
|
I'm on maven 3.3.9 and license-maven-plugin 1.12 |
For anyone just finding this now and running into a similar problem where <plugin>
<groupId>org.codehaus.mojo</groupId>
<artifactId>license-maven-plugin</artifactId>
<version>1.14</version>
<dependencies>
<dependency>
<groupId>org.apache.maven.doxia</groupId>
<artifactId>doxia-core</artifactId>
<version>1.6</version>
</dependency>
<dependency>
<groupId>org.apache.maven.doxia</groupId>
<artifactId>doxia-site-renderer</artifactId>
<version>1.6</version>
</dependency>
</dependencies>
</plugin> |
We get the folloing stacktrace after adding the license-maven-plugin as
I added doxia-core and doxia-site-renderer as jacobwil described. Any ideas? Thanks! :) |
Why was this closed? |
A mistake I suppose :( I will have a look at it soon.--Enjoy and ShareTony Chemit
On nov. 9 2018, at 9:00 pm, Ian Springer <[email protected]> wrote:
Why was this closed?
—You are receiving this because you modified the open/close state.Reply to this email directly, view it on GitHub, or mute the thread.
|
I've just tried to use the latest version 1.16, and the problem still persists. It was reported on version 1.12. Please, fix it. Can you just add this dependency into the plugin's pom.xml? |
They need to update the doxia-site-renderer from 1.4 to 1.8.1 - i don't know why they are not doing it. |
With license-maven-plugin:1.16:aggregate-third-party-report, doxia 1.8 doesn't work for me. With doxia 1.8 and 1.8.1, org/apache/maven/doxia/sink/SinkEventAttributeSet is missing (probably was moved to a new package org.apache.maven.doxia.sink.impl in 1.8). |
A pull request with an integration test is welcome. |
…report Cherry-picked and reworded from mojohaus#215 by @digulla
Fix #73 Class not found on mvn license:aggregate-third-party-report
When I try to run
mvn license:aggregate-third-party-report
I get a build failure, with the following error:Failed to execute goal org.codehaus.mojo:license-maven-plugin:1.12:aggregate-third-party-report (default-cli) on project ing-securities: Execution default-cli of goal org.codehaus.mojo:license-maven-plugin:1.12:aggregate-third-party-report failed: A required class was missing while executing org.codehaus.mojo:license-maven-plugin:1.12:aggregate-third-party-report: org/apache/maven/doxia/sink/render/RenderingContext
This is probably the same issue as #69.
That issue was closed, but I think there still is something wrong here. I downloaded the doxia jars from maven central (http://search.maven.org/#search%7Cgav%7C1%7Cg%3A%22org.apache.maven.doxia%22%20AND%20a%3A%22doxia-core%22) and starting with version 1.5, the file org/apache/maven/doxia/sink/render/RenderingContext.class is not available anymore in the jar.
I was able to to get it working by adding this to my plugin definition:
Here is a part of the stacktrace:
The text was updated successfully, but these errors were encountered: