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

[MSITE-1000] Introduce parser configuration parameter #177

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

kwin
Copy link
Member

@kwin kwin commented Mar 20, 2024

This leverages PlexusConfigurator under the hood

Requires apache/maven-doxia-sitetools#140.

@kwin kwin requested a review from michael-o March 20, 2024 16:11
Copy link
Member

@michael-o michael-o left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No logical objections, but I guess an IT to cover it up would be reasonable.

@michael-o michael-o self-requested a review April 19, 2024 19:09
Copy link
Member

@michael-o michael-o left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It does not compile for me:

[ERROR] Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:3.11.0:compile (default-compile) on project maven-site-plugin: Compilation failure: Compilation failure:
[ERROR] /var/osipovmi/Projekte/maven-site-plugin/src/main/java/org/apache/maven/plugins/site/render/ParserConfiguratorImpl.java:[35,43] cannot find symbol
[ERROR]   symbol:   class ParserConfigurator
[ERROR]   location: package org.apache.maven.doxia.siterenderer
[ERROR] /var/osipovmi/Projekte/maven-site-plugin/src/main/java/org/apache/maven/plugins/site/render/ParserConfiguratorImpl.java:[48,48] cannot find symbol
[ERROR]   symbol: class ParserConfigurator
[ERROR] /var/osipovmi/Projekte/maven-site-plugin/src/main/java/org/apache/maven/plugins/site/render/AbstractSiteRenderingMojo.java:[40,43] cannot find symbol
[ERROR]   symbol:   class ParserConfigurator
[ERROR]   location: package org.apache.maven.doxia.siterenderer
[ERROR] /var/osipovmi/Projekte/maven-site-plugin/src/main/java/org/apache/maven/plugins/site/render/AbstractSiteRenderingMojo.java:[311,78] cannot find symbol
[ERROR]   symbol:   class ParserConfigurator
[ERROR]   location: class org.apache.maven.plugins.site.render.AbstractSiteRenderingMojo
[ERROR] /var/osipovmi/Projekte/maven-site-plugin/src/main/java/org/apache/maven/plugins/site/render/SiteMojo.java:[32,43] cannot find symbol
[ERROR]   symbol:   class ParserConfigurator
[ERROR]   location: package org.apache.maven.doxia.siterenderer
[ERROR] /var/osipovmi/Projekte/maven-site-plugin/src/main/java/org/apache/maven/plugins/site/render/SiteMojo.java:[132,13] cannot find symbol
[ERROR]   symbol:   class ParserConfigurator
[ERROR]   location: class org.apache.maven.plugins.site.render.SiteMojo

@kwin kwin force-pushed the feature/configure-parser-with-plexus-configurator branch 2 times, most recently from 58db21e to 0c82571 Compare April 29, 2024 17:22
@michael-o
Copy link
Member

I will happily review again after #185 has been merged.

@michael-o michael-o self-requested a review May 1, 2024 07:38
Copy link
Member

@michael-o michael-o left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Invoking:

D:\Entwicklung\Projekte\maven-site-plugin [feature/configure-parser-with-plexus-configurator ↓1 ↑6]> mvn clean
 verify -Prun-its site -Preporting

causes

[INFO] ------------------------------------------------------------------------
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-site-plugin:3.12.1:site (default-site) on project maven-site-plugin: Error generating maven-plugin-report-plugin:3.10.2:report report: Illegal char <<> at index 22: apidocs/java/util/List<org/codehaus/plexus/configuration/PlexusConfiguration>.html -> [Help 1]

@slawekjaranowski WDYT?

@michael-o
Copy link
Member

It also fails with: Error generating maven-plugin-report-plugin:3.12.0:report report:

@michael-o
Copy link
Member

The bug is in org.apache.maven.tools.plugin.generator.PluginDescriptorFilesGenerator.getJavadocUrlForType(JavadocLinkGenerator, String). I guess it should until all generics are resolved.

@michael-o
Copy link
Member

Here is a fix: apache/maven-plugin-tools#281

@asfgit asfgit force-pushed the feature/configure-parser-with-plexus-configurator branch from 0c82571 to d5312d3 Compare May 4, 2024 17:00
This leverages PlexusConfigurator under the hood
@asfgit asfgit force-pushed the feature/configure-parser-with-plexus-configurator branch from d5312d3 to edac4a0 Compare May 5, 2024 19:24
@michael-o michael-o self-requested a review May 5, 2024 19:24
@michael-o michael-o removed their request for review November 20, 2024 15:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants