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

Draft attempt at doxia 2 #939

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Draft attempt at doxia 2 #939

wants to merge 4 commits into from

Conversation

hazendaz
Copy link
Member

No description provided.

Copy link

@elharo elharo left a comment

Choose a reason for hiding this comment

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

maven-fluido-skin 2.0;.0 is out now. That might help.


<skin>
<groupId>org.apache.maven.skins</groupId>
<artifactId>maven-fluido-skin</artifactId>
<version>1.12.0</version>
<version>2.0.0-M11</version>
Copy link

Choose a reason for hiding this comment

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

2.0.0


<skin>
<groupId>org.apache.maven.skins</groupId>
<artifactId>maven-fluido-skin</artifactId>
<version>1.12.0</version>
<version>2.0.0-M11</version>
Copy link

Choose a reason for hiding this comment

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

2.0l.0


<skin>
<groupId>org.apache.maven.skins</groupId>
<artifactId>maven-fluido-skin</artifactId>
<version>1.12.0</version>
<version>2.0.0-M11</version>
Copy link

Choose a reason for hiding this comment

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

2.0;0


<skin>
<groupId>org.apache.maven.skins</groupId>
<artifactId>maven-fluido-skin</artifactId>
<version>1.12.0</version>
<version>2.0.0-M11</version>
Copy link

Choose a reason for hiding this comment

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

2.0.0


<skin>
<groupId>org.apache.maven.skins</groupId>
<artifactId>maven-fluido-skin</artifactId>
<version>1.12.0</version>
<version>2.0.0-M11</version>
Copy link

Choose a reason for hiding this comment

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

2.0.0

Copy link
Member Author

Choose a reason for hiding this comment

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

I have it updated locally but no luck. I think part of the problem for this project is that it wants to override reporting execute that was made final.

I'll push up fresh copy tonight with this part corrected.

@elharo
Copy link

elharo commented Nov 29, 2024

This might or might not be relevant. I tried to run mvn site:site at head (i.e. not this PR but the old Doxia) and i failed like so:

``
INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 02:17 min
[INFO] Finished at: 2024-11-29T17:31:54-05:00
[INFO] ------------------------------------------------------------------------
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-site-plugin:3.12.1:site (default-cli) on project spotbugs-maven-plugin: Execution default-cli of goal org.apache.maven.plugins:maven-site-plugin:3.12.1:site failed: An API incompatibility was encountered while executing org.apache.maven.plugins:maven-site-plugin:3.12.1:site: java.lang.NoSuchMethodError: 'java.lang.String org.codehaus.mojo.versions.AbstractVersionsReport.getOutputPath()'
[ERROR] -----------------------------------------------------
[ERROR] realm = plugin>org.apache.maven.plugins:maven-site-plugin:3.12.1
[ERROR] strategy = org.codehaus.plexus.classworlds.strategy.SelfFirstStrategy
[ERROR] urls[0] = file:/Users/elharo/.m2/repository/org/apache/maven/plugins/maven-site-plugin/3.12.1/maven-site-plugin-3.12.1.jar
[ERROR] urls[1] = file:/Users/elharo/.m2/repository/org/apache/maven/doxia/doxia-sink-api/1.12.0/doxia-sink-api-1.12.0.jar
[ERROR] urls[2] = file:/Users/elharo/.m2/repository/org/apache/maven/doxia/doxia-logging-api/1.12.0/doxia-logging-api-1.12.0.jar
[ERROR] urls[3] = file:/Users/elharo/.m2/repository/org/apache/maven/skins/maven-fluido-skin/2.0.0/maven-fluido-skin-2.0.0.jar
[ERROR] urls[4] = file:/Users/elharo/.m2/repository/org/apache/maven/reporting/maven-reporting-api/3.1.1/maven-reporting-api-3.1.1.jar
...

@elharo
Copy link

elharo commented Nov 30, 2024

I have at the very least a big clue and possibly a fix for this. On a hunch I changed the parent pom from base-parent to

  <parent>
    <groupId>org.apache.maven.plugins</groupId>
    <artifactId>maven-plugins</artifactId>
    <version>42</version>
    <relativePath />
  </parent>

and with that adjustment mvn site:site correctly generated the site. I also had to disable rat, but that's an easily addressed side issue:

mvn clean site:site -Drat.skip=true

@hazendaz
Copy link
Member Author

hazendaz commented Dec 1, 2024 via email

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