(maven) branch master updated: Update formatting of prerequisites-requirements error to improve readability

[sjaranowski]

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

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


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

commit adaf662a3dc829d5a614a5f39556dfd731c353ea
Author: Slawomir Jaranowski <s.jaranow...@gmail.com>
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;
         }
     }