[MSITE-987] Consistently log when skipping report goal execution

This closes #162
apache · Oct 29, 2023 · fbf0596 · fbf0596
1 parent 6acdc6e
commit fbf0596
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git 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
@@ -225,12 +225,14 @@ protected List<MavenReportExecution> getReports() throws MojoExecutionException
         // 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);
             }