This is an automated email from the ASF dual-hosted git repository.
sjaranowski pushed a commit to branch maven-3.9.x
in repository https://gitbox.apache.org/repos/asf/maven.git
The following commit(s) were added to refs/heads/maven-3.9.x by this push:
new f839cc152f Add default implementation for new method in
MavenPluginManager
f839cc152f is described below
commit f839cc152fef7d287414446355b679e9570749b4
Author: Slawomir Jaranowski <[email protected]>
AuthorDate: Fri Dec 5 18:46:16 2025 +0100
Add default implementation for new method in MavenPluginManager
MavenPluginManager can be implemented by user extension
new method in interface can brake an implementation.
We can add default implementation in interface to avoid braking change.
---
.../src/main/java/org/apache/maven/plugin/MavenPluginManager.java | 8 ++++++--
.../apache/maven/plugin/internal/DefaultMavenPluginManager.java | 6 ------
2 files changed, 6 insertions(+), 8 deletions(-)
diff --git
a/maven-core/src/main/java/org/apache/maven/plugin/MavenPluginManager.java
b/maven-core/src/main/java/org/apache/maven/plugin/MavenPluginManager.java
index 06389c2633..f2f25bca4e 100644
--- a/maven-core/src/main/java/org/apache/maven/plugin/MavenPluginManager.java
+++ b/maven-core/src/main/java/org/apache/maven/plugin/MavenPluginManager.java
@@ -74,7 +74,9 @@ MojoDescriptor getMojoDescriptor(
* @deprecated Use {@link #checkPrerequisites(PluginDescriptor)} instead.
*/
@Deprecated
- void checkRequiredMavenVersion(PluginDescriptor pluginDescriptor) throws
PluginIncompatibleException;
+ default void checkRequiredMavenVersion(PluginDescriptor pluginDescriptor)
throws PluginIncompatibleException {
+ checkPrerequisites(pluginDescriptor);
+ }
/**
* Verifies that the specified plugin's prerequisites are met.
@@ -82,7 +84,9 @@ MojoDescriptor getMojoDescriptor(
* @param pluginDescriptor The descriptor of the plugin to check, must not
be {@code null}.
* @since 3.9.12
*/
- void checkPrerequisites(PluginDescriptor pluginDescriptor) throws
PluginIncompatibleException;
+ default void checkPrerequisites(PluginDescriptor pluginDescriptor) throws
PluginIncompatibleException {
+ checkRequiredMavenVersion(pluginDescriptor);
+ }
/**
* Sets up the class realm for the specified plugin. Both the class realm
and the plugin artifacts that constitute
diff --git
a/maven-core/src/main/java/org/apache/maven/plugin/internal/DefaultMavenPluginManager.java
b/maven-core/src/main/java/org/apache/maven/plugin/internal/DefaultMavenPluginManager.java
index b3ed212385..85ae59530d 100644
---
a/maven-core/src/main/java/org/apache/maven/plugin/internal/DefaultMavenPluginManager.java
+++
b/maven-core/src/main/java/org/apache/maven/plugin/internal/DefaultMavenPluginManager.java
@@ -309,12 +309,6 @@ public void checkPrerequisites(PluginDescriptor
pluginDescriptor) throws PluginI
}
}
- @Override
- @Deprecated
- public void checkRequiredMavenVersion(PluginDescriptor pluginDescriptor)
throws PluginIncompatibleException {
- checkPrerequisites(pluginDescriptor);
- }
-
public void setupPluginRealm(
PluginDescriptor pluginDescriptor,
MavenSession session,
