(maven-artifact-plugin) branch master updated: [MARTIFACT-60] artifact:3.5.0:check-buildplan is too chatty by default

[hboutemy]

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

hboutemy pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/maven-artifact-plugin.git


The following commit(s) were added to refs/heads/master by this push:
     new 856f707  [MARTIFACT-60] artifact:3.5.0:check-buildplan is too chatty 
by default
856f707 is described below

commit 856f70744cd3588ab7e843718dee997606bd1688
Author: Gary Gregory <garydgreg...@gmail.com>
AuthorDate: Thu Apr 11 18:03:32 2024 -0400

    [MARTIFACT-60] artifact:3.5.0:check-buildplan is too chatty by default
    
    Sample output from Apache Commons CLI
    [INFO] --- artifact:3.5.2-SNAPSHOT:check-buildplan (check-buildplan) @
    commons-cli ---
    [INFO] No known issue in 20 plugins
---
 .../maven/plugins/artifact/buildinfo/CheckBuildPlanMojo.java   | 10 ++++++++--
 1 file changed, 8 insertions(+), 2 deletions(-)

diff --git 
a/src/main/java/org/apache/maven/plugins/artifact/buildinfo/CheckBuildPlanMojo.java
 
b/src/main/java/org/apache/maven/plugins/artifact/buildinfo/CheckBuildPlanMojo.java
index 1f29371..2263474 100644
--- 
a/src/main/java/org/apache/maven/plugins/artifact/buildinfo/CheckBuildPlanMojo.java
+++ 
b/src/main/java/org/apache/maven/plugins/artifact/buildinfo/CheckBuildPlanMojo.java
@@ -101,6 +101,7 @@ public class CheckBuildPlanMojo extends AbstractMojo {
         MavenExecutionPlan plan = calculateExecutionPlan();
 
         Set<String> plugins = new HashSet<>();
+        int okCount = 0;
         for (MojoExecution exec : plan.getMojoExecutions()) {
             Plugin plugin = exec.getPlugin();
             String id = plugin.getId();
@@ -109,7 +110,8 @@ public class CheckBuildPlanMojo extends AbstractMojo {
                 // check reproducibility status
                 String issue = issues.getProperty(plugin.getKey());
                 if (issue == null) {
-                    getLog().info("no known issue with " + id);
+                    okCount++;
+                    getLog().debug("No known issue with " + id);
                 } else if (issue.startsWith("fail:")) {
                     String logMessage = "plugin without solution " + id + ", 
see " + issue.substring(5);
                     if (failOnNonReproducible) {
@@ -131,11 +133,15 @@ public class CheckBuildPlanMojo extends AbstractMojo {
                         }
                         fail = true;
                     } else {
-                        getLog().info("no known issue with " + id + " (>= " + 
issue + ")");
+                        okCount++;
+                        getLog().debug("No known issue with " + id + " (>= " + 
issue + ")");
                     }
                 }
             }
         }
+        if (okCount > 0) {
+            getLog().info("No known issue in " + okCount + " plugins");
+        }
 
         if (fail) {
             getLog().info("current module pom.xml is " + project.getBasedir() 
+ "/pom.xml");