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); }