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

Update JDK version from 1.8 to 17 #1088

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

mpsijm
Copy link

@mpsijm mpsijm commented Jan 8, 2022

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

@legendkiller1
Copy link

Can someone that knows coding take this app and update it for 1.18 and not let it die?

@jesse-savary
Copy link

@legendkiller1 I intend to fork and update this project; no ETA as of yet

@willies952002 willies952002 mentioned this pull request Mar 5, 2022
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>
Copy link
Contributor

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

Copy link
Author

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 🙂

@AwwwIgtg
Copy link

So is there a fork or whatever for 1.18?

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.

6 participants