kwin commented on pull request #681:
URL: https://github.com/apache/maven/pull/681#issuecomment-1048943328


   > Do you think that the impl in Resolver is correct?
   
   yes, but you have to keep in mind how its merge is called 
(https://github.com/apache/maven/blob/69d6c6d5a2886bd76fa9d0e922677d4654d1d90b/maven-resolver-provider/src/main/java/org/apache/maven/repository/internal/MavenMetadata.java#L75):
 The arguments are inverted to the merge from Metadata. For the former the 
destination is given as argument while for the latter the source is given as 
argument.


-- 
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: issues-unsubscr...@maven.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to