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

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

commit fbf0596c888d8fa973b42ef11b160ea04cd1c6ba
Author: Michael Osipov <micha...@apache.org>
AuthorDate: Sun Oct 29 19:11:07 2023 +0100

    [MSITE-987] Consistently log when skipping report goal execution
    
    This closes #162
---
 pom.xml                                                               | 2 +-
 .../apache/maven/plugins/site/render/AbstractSiteRenderingMojo.java   | 4 +++-
 2 files changed, 4 insertions(+), 2 deletions(-)

diff --git a/pom.xml b/pom.xml
index 9d004691..a2dc6f51 100644
--- a/pom.xml
+++ b/pom.xml
@@ -225,7 +225,7 @@ under the License.
     <dependency>
       <groupId>org.apache.maven.reporting</groupId>
       <artifactId>maven-reporting-exec</artifactId>
-      <version>2.0.0-M11</version>
+      <version>2.0.0-M12-SNAPSHOT</version>
     </dependency>
     <dependency>
       <groupId>org.apache.maven.shared</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 76bcfac5..90f60165 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
@@ -225,12 +225,14 @@ public abstract class AbstractSiteRenderingMojo extends 
AbstractSiteDescriptorMo
         // filter out reports that can't be generated
         List<MavenReportExecution> reportExecutions = new 
ArrayList<>(allReports.size());
         for (MavenReportExecution exec : allReports) {
+            String reportMojoInfo = exec.getPlugin().getId() + ":" + 
exec.getGoal();
             try {
                 if (exec.canGenerateReport()) {
                     reportExecutions.add(exec);
+                } else if (exec.isUserDefined()) {
+                    getLog().info("Skipping " + reportMojoInfo + " report");
                 }
             } catch (MavenReportException e) {
-                String reportMojoInfo = exec.getPlugin().getId() + ":" + 
exec.getGoal();
                 throw new MojoExecutionException(
                         "Failed to determine whether report '" + 
reportMojoInfo + "' can be generated", e);
             }

Reply via email to