Split the two

apache · Dec 19, 2024 · c3007ad · c3007ad
1 parent f469391
commit c3007ad
Showing 1 changed file with 11 additions and 8 deletions.
diff --git a/...ore/src/main/java/org/apache/maven/plugin/internal/DefaultPluginDependenciesResolver.java b/...ore/src/main/java/org/apache/maven/plugin/internal/DefaultPluginDependenciesResolver.java
@@ -232,14 +232,17 @@ private DependencyResult resolveInternal(
                 org.eclipse.aether.graph.Dependency managedDep =
                         core.get(pluginDep.getArtifact().getGroupId() + ":"
                                 + pluginDep.getArtifact().getArtifactId());
-                if (managedDep != null
-                        && !Objects.equals(
-                                pluginDep.getArtifact().getVersion(),
-                                managedDep.getArtifact().getVersion())) {
-                    pluginDep = pluginDep.setArtifact(pluginDep
-                            .getArtifact()
-                            .setVersion(managedDep.getArtifact().getVersion()));
-                    pluginDep = pluginDep.setScope(DependencyScope.PROVIDED.id());
+                if (managedDep != null) {
+                    // align version if needed
+                    if (!Objects.equals(
+                            pluginDep.getArtifact().getVersion(),
+                            managedDep.getArtifact().getVersion())) {
+                        pluginDep = pluginDep.setArtifact(pluginDep
+                                .getArtifact()
+                                .setVersion(managedDep.getArtifact().getVersion()));
+                    }
+                    // align scope
+                    pluginDep = pluginDep.setScope(managedDep.getScope());
                 }
                 request.addDependency(pluginDep);
             }