-
Notifications
You must be signed in to change notification settings - Fork 40.8k
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
Document that spring-boot-starter-parent sets maven.compiler.release and how to unset it if needed #37993
Comments
Thanks for trying to RC. Have you only just started seeing this problem? We started setting |
I previously discovered this PR #34365, but it still caused my program to not compile properly, and currently I have not found a corresponding solution |
Yes, I suspect that #34365 introduced the problem. However, if that is the case, this would also fail with Spring Boot 3.1. Does it? |
@wilkinsona I tried using Spring Boot version 3.1 and found the same problem |
Thanks. You can undo <maven.compiler.release></maven.compiler.release> You then have to configure the source and target instead: <maven.compiler.source>${java.version}</maven.compiler.source>
<maven.compiler.target>${java.version}</maven.compiler.target> With these three properties in place, a project that uses We'll have to decide if we want to continue to set |
Thank you very much. |
--add-exports
compiler argument
Upside of |
It's definitely a good thing if you're building with a JDK version other than the one with which you'll run your app. However, if the two match, that benefit disappears and I wonder how common it is for them not to match.
I don't. |
--add-exports
compiler argument
We're going to document that we set |
springboot 3.2.0-RC1
the default property
<maven.compiler.release>${java.version}</maven.compiler.release>
cause error
exporting a package from system module java.base is not allowed with --release
see
source
plexus-compiler-javac-2.13.0.jar
The text was updated successfully, but these errors were encountered: