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

sjaranowski pushed a commit to branch maven-4.0.x
in repository https://gitbox.apache.org/repos/asf/maven.git


The following commit(s) were added to refs/heads/maven-4.0.x by this push:
     new a20e3a1a30 Update formatting of prerequisites-requirements error to 
improve readability
a20e3a1a30 is described below

commit a20e3a1a3080bb5a905c9f6f8dc12d8b54213cdb
Author: Slawomir Jaranowski <[email protected]>
AuthorDate: Sat Dec 6 16:55:53 2025 +0100

    Update formatting of prerequisites-requirements error to improve readability
    
    Message at the end of error line is not readable ... so add some new lines 
and tabs to make it more readable
    
    chery pick from ec21f4bf223c3f2e5fda052d405e2b432fea8e7b
---
 .../org/apache/maven/plugin/MavenPluginPrerequisitesChecker.java  | 2 +-
 .../apache/maven/plugin/internal/DefaultMavenPluginManager.java   | 8 +++-----
 2 files changed, 4 insertions(+), 6 deletions(-)

diff --git 
a/impl/maven-core/src/main/java/org/apache/maven/plugin/MavenPluginPrerequisitesChecker.java
 
b/impl/maven-core/src/main/java/org/apache/maven/plugin/MavenPluginPrerequisitesChecker.java
index ee240159ae..7dd84bb674 100644
--- 
a/impl/maven-core/src/main/java/org/apache/maven/plugin/MavenPluginPrerequisitesChecker.java
+++ 
b/impl/maven-core/src/main/java/org/apache/maven/plugin/MavenPluginPrerequisitesChecker.java
@@ -29,7 +29,7 @@
 public interface MavenPluginPrerequisitesChecker extends 
Consumer<PluginDescriptor> {
     /**
      *
-     * @param pluginDescriptor
+     * @param pluginDescriptor the plugin descriptor to check
      * @throws IllegalStateException in case the checked prerequisites are not 
met
      */
     @Override
diff --git 
a/impl/maven-core/src/main/java/org/apache/maven/plugin/internal/DefaultMavenPluginManager.java
 
b/impl/maven-core/src/main/java/org/apache/maven/plugin/internal/DefaultMavenPluginManager.java
index 93c0d5a123..e51b4dd0dd 100644
--- 
a/impl/maven-core/src/main/java/org/apache/maven/plugin/internal/DefaultMavenPluginManager.java
+++ 
b/impl/maven-core/src/main/java/org/apache/maven/plugin/internal/DefaultMavenPluginManager.java
@@ -316,13 +316,11 @@ public void checkPrerequisites(PluginDescriptor 
pluginDescriptor) throws PluginI
         if (!prerequisiteExceptions.isEmpty()) {
             String messages = prerequisiteExceptions.stream()
                     .map(IllegalStateException::getMessage)
-                    .collect(Collectors.joining(", "));
+                    .collect(Collectors.joining("\n\t"));
             PluginIncompatibleException pie = new PluginIncompatibleException(
                     pluginDescriptor.getPlugin(),
-                    "The plugin " + pluginDescriptor.getId() + " has unmet 
prerequisites: " + messages,
-                    prerequisiteExceptions.get(0));
-            // the first exception is added as cause, all other ones as 
suppressed exceptions
-            
prerequisiteExceptions.stream().skip(1).forEach(pie::addSuppressed);
+                    "\nThe plugin " + pluginDescriptor.getId() + " has unmet 
prerequisites: \n\t" + messages);
+            prerequisiteExceptions.forEach(pie::addSuppressed);
             throw pie;
         }
     }

Reply via email to