Skip to content
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

[MNG-8368] Fix dependency resolver not using project repositories #1865

Merged
merged 1 commit into from
Nov 5, 2024

Conversation

gnodet
Copy link
Contributor

@gnodet gnodet commented Nov 4, 2024

JIRA issue: MNG-8368

@gnodet gnodet added this to the 4.0.0-beta-6 milestone Nov 5, 2024
@desruisseaux
Copy link
Contributor

I confirm that with this fix, all the integration tests of the new compiler plugin pass. Tested with both Java 17 and 23.

@gnodet gnodet changed the title Fix dependency resolver not using project repositories [MNG-8368] Fix dependency resolver not using project repositories Nov 5, 2024
@gnodet gnodet merged commit 62c94d1 into apache:master Nov 5, 2024
13 checks passed
desruisseaux added a commit to Geomatys/maven-compiler-plugin that referenced this pull request Nov 5, 2024
It will be re-enabled after Maven 4.0.0-beta-6 release.

apache/maven#1865
gnodet pushed a commit to apache/maven-compiler-plugin that referenced this pull request Nov 14, 2024
#271)

* Rewrite the plugin using standard `javax.tools` API instead of Plexus.
Other aspects that are rewritten include incremental build mechanism
and how the overwriting of `module-info.java` in tests is handled.
For details and impact on users, see pull request description at
#271

* Temporarily disable the MCOMPILER-346 integration test.
It will be re-enabled after Maven 4.0.0-beta-6 release.

apache/maven#1865

* Remove the replacement of `/` by `\` on the Windows platform in GLOB syntax.
It appears that the GLOB matcher expects `/` even on Windows.

* Add a log at the error level when the compiler threw an exception instead of returning the Boolean `false` value.
Adjust what is the cause and what is the suppressed exception.

* Set the `maven-plugin-testing-harness` version to 4.0.0-beta-2.

* Dummy change for forcing a new CI build.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants