-
Notifications
You must be signed in to change notification settings - Fork 227
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
breaks on JDK 13 #154
Comments
Retrolambda only supports backporting Java 8 bytecode. Backporting newer language features is outside the scope of this project, so the workaround is to compile the project using Java 8. |
The source is 1.8 on this project (ex set in the compiler plugin) and all is well in JDK 11. Are you saying that JDK 11 accidentally works and that we shouldn't expect it to? PS we also have fork set to true here |
Retrolambda's tests are run using also JDK 11 (https://github.com/luontola/retrolambda/blob/master/scripts/build.sh#L82), so it works to some extend, though features beyond Java 8 are not backported. It could still break at some point, so building the project with Java 8 is the best bet. The error "Unsupported class file major version 57" indicates that some classes use Java 13 bytecode. The IllegalAccessException might have to do with stricter access checks in the recent Java versions. |
Upgrading ASM library fixed the Java 13 compatibility. Java 12 and 13 should work only with fork=true. This fix is included in Retrolambda 2.5.7. |
all our zipkin are belong to you
thanks so much for the help!
|
Not sure if this project is active anymore, but would be awesome if it were. Currently JDK 13 isn't usable due to below:
The text was updated successfully, but these errors were encountered: