Skip to content

Resolver 2.0.12 #11137

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 14 commits into
base: master
Choose a base branch
from
Open

Resolver 2.0.12 #11137

wants to merge 14 commits into from
+68 −17

Conversation

cstamas
Copy link
Member

@cstamas cstamas commented Sep 17, 2025 •
edited
Loading

Just a PR to have it prepared here. Resolver 2.0.12 has fixed several bugs and issues from 2.0.11 causing endless loops/stack overflow with some more complex graphs.

Changes in ITs:

  • JDK transport is "demoted" (default is again "apache")
  • classpath string is now levelOrder (not preOrder as Maven 3)

@cstamas
Resolver 2.0.12
3095972 
Just a PR to have it prepped here. Resolver 2.0.11 has identified
several issues causing endless loops/stack overflow with some
more complex graphs.
@cstamas cstamas added this to the 4.0.0 milestone Sep 17, 2025
@cstamas cstamas self-assigned this Sep 17, 2025
@cstamas cstamas added the dependencies Pull requests that update a dependency file label Sep 17, 2025
@cstamas
Copy link
Member Author

cstamas commented Sep 20, 2025

As expected:

[INFO] Results:
[INFO] 
Error:  Failures: 
Error:    MavenITmng3813PluginClassPathOrderingTest.testitMNG3813:72 expected: <true> but was: <false>
[INFO] 
Error:  Tests run: 989, Failures: 1, Errors: 0, Skipped: 51

due classpath ordering change. woohoo!

@cstamas cstamas marked this pull request as ready for review September 29, 2025 11:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gnodet gnodet Awaiting requested review from gnodet

@slawekjaranowski slawekjaranowski Awaiting requested review from slawekjaranowski

Assignees

@cstamas cstamas

Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Milestone
4.0.0
Development

Successfully merging this pull request may close these issues.
1 participant
@cstamas