Skip to content

[MNG-8590] Make sure maven-resolver-provider does not depend on maven-impl#2123

Merged
gnodet merged 4 commits intoapache:masterfrom
gnodet:remove-maven-impl-dep
Feb 25, 2025
Merged

[MNG-8590] Make sure maven-resolver-provider does not depend on maven-impl#2123
gnodet merged 4 commits intoapache:masterfrom
gnodet:remove-maven-impl-dep

Conversation

@gnodet
Copy link
Contributor

@gnodet gnodet commented Feb 24, 2025

JIRA issue: MNG-8590

In order to be able to use the maven-resolver-provider as it used to be (see MNG-8583), we need to break the dependency from maven-resolver-provider to maven-impl so that we don't have duplicate bean definitions.

Related to #2110

@gnodet gnodet force-pushed the remove-maven-impl-dep branch from a5b8d3e to 616549a Compare February 24, 2025 20:58
@gnodet gnodet changed the title Make sure maven-resolver-prodiverdoes not depend on maven-impl Make sure maven-resolver-prodiver does not depend on maven-impl Feb 24, 2025
@gnodet gnodet changed the title Make sure maven-resolver-prodiver does not depend on maven-impl Make sure maven-resolver-provider does not depend on maven-impl Feb 24, 2025
@gnodet gnodet force-pushed the remove-maven-impl-dep branch from bdfc2d3 to 1ec6dcb Compare February 24, 2025 21:57
@gnodet gnodet force-pushed the remove-maven-impl-dep branch from 1ec6dcb to 73245a8 Compare February 24, 2025 22:13
@gnodet gnodet self-assigned this Feb 25, 2025
@gnodet gnodet added this to the 4.0.0-rc-3 milestone Feb 25, 2025
@gnodet gnodet marked this pull request as ready for review February 25, 2025 06:05
@gnodet gnodet requested a review from cstamas February 25, 2025 06:05
@gnodet gnodet changed the title Make sure maven-resolver-provider does not depend on maven-impl [MNG-8590] Make sure maven-resolver-provider does not depend on maven-impl Feb 25, 2025
@gnodet gnodet merged commit 006ad92 into apache:master Feb 25, 2025
13 checks passed
@jira-importer
Copy link

Resolve #9855

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants