cstamas commented on PR #765: URL: https://github.com/apache/maven/pull/765#issuecomment-1183234729
Let's move this discussion to ML. --- Regarding this PR: is wrong, and the reason is that _provided scoped dependencies_ are not present in list I am checking (nor in DependencyNode root) as they are filtered out way before (buried in resolver). Hence, is easy to trick out this check: use **latest** mpp and set **provided** scope for maven-plugin-api having version **2.2.1** :smile: and this code will not trigger (as provided scope is not seen). We need a bit of different approach, and probably we need to grab plugin POM (not happening currently, only as "side effect" of resolving the plugin artifact) as that would be the simplest, and is already resolved (so is in local repo). -- 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
