-
Notifications
You must be signed in to change notification settings - Fork 238
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
Update JDK version from 1.8 to 17 #1088
base: master
Are you sure you want to change the base?
Conversation
Can someone that knows coding take this app and update it for 1.18 and not let it die? |
@legendkiller1 I intend to fork and update this project; no ETA as of yet |
pom.xml
Outdated
@@ -56,7 +56,7 @@ | |||
<plugin> | |||
<groupId>org.apache.maven.plugins</groupId> | |||
<artifactId>maven-shade-plugin</artifactId> | |||
<version>3.2.4</version> | |||
<version>3.3.0-SNAPSHOT</version> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this should just be 3.3.0
, no -SNAPSHOT
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the heads-up! 😄 That version seems to have been released about a month ago 🙂
So is there a fork or whatever for 1.18? |
Supersedes #1020 (thanks @Mobmaker55 for making a start with this! 🙂)
This PR updates the build configuration to use Java 17. This makes Amidst work for Minecraft 1.17 and below.
In Minecraft 1.18, world generation has had a large overhaul, and I am not planning to make an attempt to update Amidst to work with this (I have experience with Java, but not with Minecraft internals, unfortunately).
For anyone who wants to use the Java 17 version of Amidst v4.7, I have published the binaries here: https://github.com/mpsijm/amidst/releases/tag/v4.7-java-17