-
Notifications
You must be signed in to change notification settings - Fork 2.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[3.8.x][MNG-7349] Limit relocation warning message to direct dependencies only #657
Conversation
This reverts commit a1ba330.
maven-core/src/main/java/org/apache/maven/project/DefaultProjectDependenciesResolver.java
Outdated
Show resolved
Hide resolved
If you plan to add an IT for that, I have recently added a test for relocated plugins and their config. You can reuse (copy) that and test it. |
Stupid question: Will it also apply to extensions? |
Yes, both boot and build extensions use the |
...n-core/src/main/java/org/apache/maven/plugin/internal/DefaultPluginDependenciesResolver.java
Outdated
Show resolved
Hide resolved
...-resolver-provider/src/main/java/org/apache/maven/repository/internal/RelocatedArtifact.java
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The code is just fine, only the IT needs refinement then I am fine to squash and merge into master and maven-3.8.x.
JIRA issue: https://issues.apache.org/jira/browse/MNG-7349
IT: apache/maven-integration-testing#134