You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by mi...@apache.org on 2022/04/15 19:00:52 UTC

[maven-site-plugin] branch doxia-2.0.0 updated (60675095 -> 76e2dc80)

This is an automated email from the ASF dual-hosted git repository.

michaelo pushed a change to branch doxia-2.0.0
in repository https://gitbox.apache.org/repos/asf/maven-site-plugin.git


 discard 60675095 first shot
     add 6c53dfc1 Shared GitHub actions v2
     add 0b1c775f [MSITE-829] [MSITE-828] Upgrade Jetty to 9.4.x implicit java8 requirement now (#21)
     add 0d5299be Enable release drafter (#75)
     add 13787944 Bump mrm-maven-plugin from 1.2.0 to 1.3.0
     add af097dce Bump slf4jVersion from 1.7.32 to 1.7.36
     new 76e2dc80 first shot

This update added new revisions after undoing existing revisions.
That is to say, some revisions that were in the old version of the
branch are not in the new version.  This situation occurs
when a user --force pushes a change and generates a repository
containing something like this:

 * -- * -- B -- O -- O -- O   (60675095)
            \
             N -- N -- N   refs/heads/doxia-2.0.0 (76e2dc80)

You should already have received notification emails for all of the O
revisions, and so the following emails describe only the N revisions
from the common base, B.

Any revisions marked "omit" are not gone; other references still
refer to them.  Any revisions marked "discard" are gone forever.

The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.


Summary of changes:
 .../release-drafter.yml                              |  5 +++--
 .github/workflows/maven-verify.yml                   |  6 +-----
 .../{maven-verify.yml => release-drafter.yml}        | 20 ++++++++------------
 Jenkinsfile                                          |  2 +-
 pom.xml                                              |  8 +++++---
 src/it/projects/MSITE-497/pom.xml                    |  1 +
 6 files changed, 19 insertions(+), 23 deletions(-)
 copy src/it/projects/MSITE-609/invoker.properties => .github/release-drafter.yml (84%)
 copy .github/workflows/{maven-verify.yml => release-drafter.yml} (81%)


[maven-site-plugin] 01/01: first shot

Posted by mi...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

michaelo pushed a commit to branch doxia-2.0.0
in repository https://gitbox.apache.org/repos/asf/maven-site-plugin.git

commit 76e2dc80444b8ab3902deb0bc5214d2134eca2bf
Author: Michael Osipov <mi...@apache.org>
AuthorDate: Sun Feb 20 21:36:15 2022 +0100

    first shot
---
 pom.xml                                            | 27 ++-----------
 .../site/render/AbstractSiteRenderingMojo.java     | 45 ++++++++--------------
 .../render/CategorySummaryDocumentRenderer.java    |  8 +---
 .../site/render/ReportDocumentRenderer.java        |  3 --
 .../apache/maven/plugins/site/run/DoxiaFilter.java |  2 +-
 5 files changed, 21 insertions(+), 64 deletions(-)

diff --git a/pom.xml b/pom.xml
index 338988d8..f5e1a84a 100644
--- a/pom.xml
+++ b/pom.xml
@@ -195,14 +195,12 @@ under the License.
   </distributionManagement>
 
   <properties>
-    <mavenVersion>3.0.5</mavenVersion>
+    <mavenVersion>3.2.5</mavenVersion>
     <javaVersion>8</javaVersion>
-
     <!-- for dependencies -->
-    <doxiaVersion>1.11.1</doxiaVersion>
-    <doxiaSitetoolsVersion>1.11.1</doxiaSitetoolsVersion>
+    <doxiaVersion>2.0.0-M2</doxiaVersion>
+    <doxiaSitetoolsVersion>2.0.0-M2</doxiaSitetoolsVersion>
     <jettyVersion>9.4.44.v20210927</jettyVersion>
-
     <plexusVersion>1.5.4</plexusVersion>
     <scmVersion>1.4</scmVersion>
     <wagonVersion>3.3.1</wagonVersion>
@@ -369,24 +367,7 @@ under the License.
       <version>${doxiaVersion}</version>
       <scope>runtime</scope>
     </dependency>
-    <dependency>
-      <groupId>org.apache.maven.doxia</groupId>
-      <artifactId>doxia-module-confluence</artifactId>
-      <version>${doxiaVersion}</version>
-      <scope>runtime</scope>
-    </dependency>
-    <dependency>
-      <groupId>org.apache.maven.doxia</groupId>
-      <artifactId>doxia-module-docbook-simple</artifactId>
-      <version>${doxiaVersion}</version>
-      <scope>runtime</scope>
-    </dependency>
-    <dependency>
-      <groupId>org.apache.maven.doxia</groupId>
-      <artifactId>doxia-module-twiki</artifactId>
-      <version>${doxiaVersion}</version>
-      <scope>runtime</scope>
-    </dependency>
+
     <!-- Doxia Sitetools -->
     <dependency>
       <groupId>org.apache.maven.doxia</groupId>
diff --git a/src/main/java/org/apache/maven/plugins/site/render/AbstractSiteRenderingMojo.java b/src/main/java/org/apache/maven/plugins/site/render/AbstractSiteRenderingMojo.java
index 16398c85..071d8a4b 100644
--- a/src/main/java/org/apache/maven/plugins/site/render/AbstractSiteRenderingMojo.java
+++ b/src/main/java/org/apache/maven/plugins/site/render/AbstractSiteRenderingMojo.java
@@ -300,40 +300,25 @@ public abstract class AbstractSiteRenderingMojo extends AbstractSiteDescriptorMo
         }
 
         SiteRenderingContext context;
-        if ( templateFile != null )
+        try
         {
-            getLog().info( buffer().a( "Rendering content with " ).strong( templateFile
-                + " template file" ).a( '.' ).toString() );
+           Artifact skinArtifact =
+               siteTool.getSkinArtifactFromRepository( localRepository, repositories, decorationModel );
 
-            if ( !templateFile.exists() )
-            {
-                throw new MojoFailureException( "Template file '" + templateFile + "' does not exist" );
-            }
-            context = siteRenderer.createContextForTemplate( templateFile, attributes, decorationModel,
-                                                             project.getName(), locale );
+            getLog().info( buffer().a( "Rendering content with " ).strong( skinArtifact.getId()
+                + " skin" ).a( '.' ).toString() );
+
+            context = siteRenderer.createContextForSkin( skinArtifact, attributes, decorationModel,
+                                                         project.getName(), locale );
         }
-        else
+        catch ( SiteToolException e )
         {
-            try
-            {
-                Artifact skinArtifact =
-                    siteTool.getSkinArtifactFromRepository( localRepository, repositories, decorationModel );
-
-                getLog().info( buffer().a( "Rendering content with " ).strong( skinArtifact.getId()
-                    + " skin" ).a( '.' ).toString() );
-
-                context = siteRenderer.createContextForSkin( skinArtifact, attributes, decorationModel,
-                                                             project.getName(), locale );
-            }
-            catch ( SiteToolException e )
-            {
-                throw new MojoExecutionException( "SiteToolException while preparing skin: " + e.getMessage(), e );
-            }
-            catch ( RendererException e )
-            {
-                throw new MojoExecutionException( "RendererException while preparing context for skin: "
-                    + e.getMessage(), e );
-            }
+            throw new MojoExecutionException( "SiteToolException while preparing skin: " + e.getMessage(), e );
+        }
+        catch ( RendererException e )
+        {
+            throw new MojoExecutionException( "RendererException while preparing context for skin: "
+                + e.getMessage(), e );
         }
 
         // Generate static site
diff --git a/src/main/java/org/apache/maven/plugins/site/render/CategorySummaryDocumentRenderer.java b/src/main/java/org/apache/maven/plugins/site/render/CategorySummaryDocumentRenderer.java
index e2979f03..9c295a10 100644
--- a/src/main/java/org/apache/maven/plugins/site/render/CategorySummaryDocumentRenderer.java
+++ b/src/main/java/org/apache/maven/plugins/site/render/CategorySummaryDocumentRenderer.java
@@ -32,7 +32,6 @@ import org.apache.maven.doxia.siterenderer.RendererException;
 import org.apache.maven.doxia.siterenderer.RenderingContext;
 import org.apache.maven.doxia.siterenderer.SiteRenderingContext;
 import org.apache.maven.doxia.siterenderer.sink.SiteRendererSink;
-import org.apache.maven.doxia.tools.MojoLogWrapper;
 import org.apache.maven.plugin.logging.Log;
 import org.apache.maven.reporting.MavenReport;
 import org.codehaus.plexus.i18n.I18N;
@@ -82,11 +81,6 @@ public class CategorySummaryDocumentRenderer
     {
         SiteRendererSink sink = new SiteRendererSink( renderingContext );
 
-        if ( log != null )
-        {
-            sink.enableLogging( new MojoLogWrapper( log ) );
-        }
-
         sink.head();
 
         sink.title();
@@ -172,7 +166,7 @@ public class CategorySummaryDocumentRenderer
 
         sink.close();
 
-        renderer.generateDocument( writer, sink, siteRenderingContext );
+        renderer.mergeDocumentIntoSite( writer, sink, siteRenderingContext );
     }
 
     public String getOutputName()
diff --git a/src/main/java/org/apache/maven/plugins/site/render/ReportDocumentRenderer.java b/src/main/java/org/apache/maven/plugins/site/render/ReportDocumentRenderer.java
index 45cf613f..40cf3ad0 100644
--- a/src/main/java/org/apache/maven/plugins/site/render/ReportDocumentRenderer.java
+++ b/src/main/java/org/apache/maven/plugins/site/render/ReportDocumentRenderer.java
@@ -40,7 +40,6 @@ import org.apache.maven.doxia.siterenderer.RendererException;
 import org.apache.maven.doxia.siterenderer.RenderingContext;
 import org.apache.maven.doxia.siterenderer.SiteRenderingContext;
 import org.apache.maven.doxia.siterenderer.sink.SiteRendererSink;
-import org.apache.maven.doxia.tools.MojoLogWrapper;
 import org.apache.maven.plugin.logging.Log;
 import org.apache.maven.reporting.MavenMultiPageReport;
 import org.apache.maven.reporting.MavenReport;
@@ -280,8 +279,6 @@ public class ReportDocumentRenderer
 
             for ( MultiPageSubSink mySink : sinks )
             {
-                mySink.enableLogging( new MojoLogWrapper( log ) );
-
                 outputName = mySink.getOutputName();
                 log.debug( "  Rendering " + outputName );
 
diff --git a/src/main/java/org/apache/maven/plugins/site/run/DoxiaFilter.java b/src/main/java/org/apache/maven/plugins/site/run/DoxiaFilter.java
index 7c698969..012ae187 100644
--- a/src/main/java/org/apache/maven/plugins/site/run/DoxiaFilter.java
+++ b/src/main/java/org/apache/maven/plugins/site/run/DoxiaFilter.java
@@ -177,7 +177,7 @@ public class DoxiaFilter
             try
             {
                 Map<String, DocumentRenderer> locateDocuments =
-                    siteRenderer.locateDocumentFiles( generatedSiteContext );
+                    siteRenderer.locateDocumentFiles( generatedSiteContext, false );
 
                 if ( locateDocuments.containsKey( path ) )
                 {