-
Notifications
You must be signed in to change notification settings - Fork 8.9k
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
HADOOP-17551. Upgrade maven-site-plugin to 3.11.0 #4196
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1 pending Jenkins
💔 -1 overall
This message was automatically generated. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
mvn site is failing: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4196/1/artifact/out/patch-mvnsite-root.txt
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-site-plugin:3.11.0:site (default-site) on project hadoop-project: Execution default-site of goal org.apache.maven.plugins:maven-site-plugin:3.11.0:site failed: An API incompatibility was encountered while executing org.apache.maven.plugins:maven-site-plugin:3.11.0:site: java.lang.AbstractMethodError: org.apache.maven.doxia.module.markdown.MarkdownParser.parse(Ljava/io/Reader;Lorg/apache/maven/doxia/sink/Sink;Ljava/lang/String;)V
I think the error is caused by the doxia upgrades such as https://issues.apache.org/jira/browse/MSITE-878. Would you remove the hard-coded doxia version in pom.xml?
Lines 316 to 320 in de2e265
<dependency> | |
<groupId>org.apache.maven.doxia</groupId> | |
<artifactId>doxia-module-markdown</artifactId> | |
<version>1.8</version> | |
</dependency> |
I removed the above lines and mvn site
succeeded on my local.
@aajisaka - I will do that. |
💔 -1 overall
This message was automatically generated. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you @ashutoshcipher
Reviewed-by: Ayush Saxena <[email protected]> Signed-off-by: Akira Ajisaka <[email protected]> (cherry picked from commit 56cfd60)
Reviewed-by: Ayush Saxena <[email protected]> Signed-off-by: Akira Ajisaka <[email protected]>
Description of PR
Upgrade maven-site-plugin to 3.11.0