From 588472cac0b85ca20b96832495a8676f97ef94b9 Mon Sep 17 00:00:00 2001 From: Slawomir Jaranowski Date: Sat, 6 Dec 2025 16:55:53 +0100 Subject: [PATCH] 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 --- .../maven/plugin/MavenPluginPrerequisitesChecker.java | 2 +- .../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 ee240159ae83..7dd84bb674aa 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 { /** * - * @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 93c0d5a1237e..e51b4dd0ddce 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; } }