Skip to content

Conversation

@cstamas
Copy link
Member

@cstamas cstamas commented Sep 17, 2025

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

Changes in ITs:

  • classpath string is now levelOrder (not preOrder as Maven 3)
  • error string comparison that fits apache and jdk transport (error message is slightly different)

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
@cstamas cstamas changed the title Resolver 2.0.12 Resolver 2.0.13 Oct 8, 2025
@cstamas cstamas merged commit 14d6d44 into apache:master Oct 10, 2025
19 checks passed
@cstamas cstamas deleted the resolver-2.0.12 branch October 10, 2025 09:40
cstamas added a commit to cstamas/maven that referenced this pull request Oct 10, 2025
Resolver 2.0.13 has fixed several bugs and issues from 2.0.11 causing endless loops/stack overflow with some more complex graphs.

Changes in ITs:
* classpath string is now levelOrder (not preOrder as Maven 3)
* error string comparison that fits apache and jdk transport (error message is slightly different)
@cstamas cstamas modified the milestones: 4.0.0, 4.1.0 Oct 10, 2025
cstamas added a commit that referenced this pull request Oct 10, 2025
Resolver 2.0.13 has fixed several bugs and issues from 2.0.11 causing endless loops/stack overflow with some more complex graphs.

Changes in ITs:
* classpath string is now levelOrder (not preOrder as Maven 3)
* error string comparison that fits apache and jdk transport (error message is slightly different)

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

Labels

dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant