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

Enable Compilation on JDK21 #4859

Merged
merged 1 commit into from
Apr 29, 2024
Merged

Conversation

rvullriede
Copy link
Contributor

The currently used lombok version is not compatible with JDK 21.
In addition, Its advised to switch from source/target to release (see https://openjdk.org/jeps/247 for details), introduced in Java 9.
The maven compiler plugin with translate this automatically to the old source/target/bootclasspath options for Java 8 compilers.

@rizer1980
Copy link
Contributor

Hello.
Maybe it’s worth separating these (OHLC for Kraken, JDK 21) for two separate PR?

@rvullriede rvullriede closed this Apr 29, 2024
@rvullriede rvullriede deleted the fix/jdk-21-support branch April 29, 2024 06:36
@rvullriede
Copy link
Contributor Author

@rizer1980, sorry that was unintentional. I'll cleaned this up. In addition, I noticed that meanwhile the problematic lombok dependency was updated already. What remains are the more robust compilation settings.

@rvullriede rvullriede reopened this Apr 29, 2024
@bigscoop
Copy link
Contributor

How about changing the code level to 11? Since we're anyway building witth SDK 11. What do you think, @timmolter ?

@timmolter timmolter merged commit 5238eab into knowm:develop Apr 29, 2024
2 checks passed
@timmolter
Copy link
Member

@bigscoop ok, let's try it after the next release (coming this week). That way Java 11 will be present in the SNAPSHOT versions for a while before releasing it. We've tried several times in the past, but people reported issues with Android. Maybe it will work out this time around.

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.

4 participants