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

Clarify Maven configuration #3045

Closed
wants to merge 1 commit into from

Conversation

anthonyvdotbe
Copy link
Contributor

The current way of using .mvn/jvm.config only works if the maven-compiler-plugin doesn't use an external executable.
The previous way of using compilerArgs only works if the maven-compiler-plugin does use an external executable.

So both ways should be documented, such that users can select the way that works for their Maven setup.
Also note that setting fork to true is just one way to get the maven-compiler-plugin to use an external executable. Enabling the maven-toolchains-plugin is another.

PS: not sure whether I should create an issue for this, or if this can go under #2786?

@cushon
Copy link
Collaborator

cushon commented Sep 12, 2023

Thanks! Merged as 175b9db

@cushon cushon closed this Sep 12, 2023
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.

2 participants