kwin commented on PR #1245: URL: https://github.com/apache/maven-site-plugin/pull/1245#issuecomment-4184146717
This is incompatible with maven-changes-plugin 2.1.12 due to its ancient classpath: ``` [DEBUG] Included: org.apache.maven.doxia:doxia-core:jar:1.4 [DEBUG] Included: org.apache.maven.doxia:doxia-logging-api:jar:1.4 [DEBUG] Included: org.codehaus.plexus:plexus-component-annotations:jar:1.6 [DEBUG] Included: org.apache.maven.doxia:doxia-module-xhtml:jar:1.4 [DEBUG] Included: org.apache.maven.doxia:doxia-decoration-model:jar:1.4 [DEBUG] Included: org.apache.maven.doxia:doxia-module-fml:jar:1.4 ``` Notably it doesn't include doxia-sink-api with its new method https://github.com/apache/maven-doxia/blob/ccfdfef76df1f4273dd8f5564ce1fa0456b59e1d/doxia-sink-api/src/main/java/org/apache/maven/doxia/sink/SinkEventAttributes.java#L412, therefore leading to ``` Warning: An issue has occurred with maven-changes-plugin:2.12.1:changes-report report, skipping LinkageError Receiver class org.apache.maven.doxia.sink.SinkEventAttributeSet does not define or inherit an implementation of the resolved method 'abstract java.util.Set entrySet()' of interface org.apache.maven.doxia.sink.SinkEventAttributes., please report an issue to Maven dev team. java.lang.AbstractMethodError: Receiver class org.apache.maven.doxia.sink.SinkEventAttributeSet does not define or inherit an implementation of the resolved method 'abstract java.util.Set entrySet()' of interface org.apache.maven.doxia.sink.SinkEventAttributes. at org.apache.maven.doxia.sink.impl.Xhtml5BaseSink.convertToHtml5CompliantAttributes (Xhtml5BaseSink.java:1765) at org.apache.maven.doxia.sink.impl.Xhtml5BaseSink.convertAndFilterAttributes (Xhtml5BaseSink.java:1748) at org.apache.maven.doxia.sink.impl.Xhtml5BaseSink.onSectionTitle (Xhtml5BaseSink.java:370) at org.apache.maven.doxia.sink.impl.Xhtml5BaseSink.sectionTitle (Xhtml5BaseSink.java:317) at org.apache.maven.plugin.changes.ChangesReportGenerator.constructRelease (ChangesReportGenerator.java:534) at org.apache.maven.plugin.changes.ChangesReportGenerator.constructReleases (ChangesReportGenerator.java:515) at org.apache.maven.plugin.changes.ChangesReportGenerator.doGenerateReport (ChangesReportGenerator.java:232) at org.apache.maven.plugin.changes.ChangesMojo.executeReport (ChangesMojo.java:347) at org.apache.maven.reporting.AbstractMavenReport.generate (AbstractMavenReport.java:255) at org.apache.maven.plugins.site.render.ReportDocumentRenderer.renderDocument (ReportDocumentRenderer.java:199) at org.apache.maven.doxia.siterenderer.DefaultSiteRenderer.render (DefaultSiteRenderer.java:302) at org.apache.maven.plugins.site.render.SiteMojo.renderNonDoxiaDocuments (SiteMojo.java:295) at org.apache.maven.plugins.site.render.SiteMojo.renderLocale (SiteMojo.java:160) at org.apache.maven.plugins.site.render.SiteMojo.execute (SiteMojo.java:125) at org.apache.maven.plugin.DefaultBuildPluginManager.executeMojo (DefaultBuildPluginManager.java:126) ``` @michael-o Do you consider this change backwards compatible enough: https://github.com/apache/maven-doxia/blob/ccfdfef76df1f4273dd8f5564ce1fa0456b59e1d/doxia-sink-api/src/main/java/org/apache/maven/doxia/sink/SinkEventAttributes.java#L412? Seems that doxia-core and doxia-sink-api should have a similar version due to the former implementing api from the latter, however one is determined from the reporting plugin while the other is determined from the Maven Site plugin which leads to this kind of issues. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
