rfscholte commented on a change in pull request #603:
URL: https://github.com/apache/maven/pull/603#discussion_r746795957
##########
File path:
maven-model-builder/src/main/java/org/apache/maven/model/composition/DefaultDependencyManagementImporter.java
##########
@@ -83,4 +90,43 @@ public void importManagement( Model target, List<? extends
DependencyManagement>
}
}
+ static void updateDependencyHierarchy( Dependency dependency,
DependencyManagement bom )
+ {
+ // We are only interested in the InputSource, so the location of the
<dependency> element is sufficient
+ InputLocation dependencyLocation = dependency.getLocation( "" );
+ InputLocation bomLocation = bom.getLocation( "" );
+
+ if ( dependencyLocation == null || bomLocation == null )
+ {
+ return;
+ }
+
+ InputSource hierarchicalSource = dependencyLocation.getSource();
Review comment:
For the effective pom there will be only one route, either `bom1` or
`parent` wins. Saying it differently: if something in bom3 has to be changed
and I have access to both bom1 and parent, I need to know which of the 2 has to
be changed.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
