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

Bump Minecraft Default Java Version Value to 21 (Ensure Compatibility with 1.20.5+) [PUFFERPANEL'S TEMURIN DOCKER IMAGE NEEDS UPDATE TO SUPPORT JDK 21 FIRST, unless it uses the original Temurin image?] #273

Merged
merged 5 commits into from
May 1, 2024

Conversation

Minionguyjpro
Copy link
Contributor

@Minionguyjpro Minionguyjpro commented May 1, 2024

This PR bumps the default Java version value for all Minecraft Java templates to Java 21, in order to increase compatibility by default. Because of this the user doesn't need to tinker, and it is required by 1.20.5 and newer. Newer Java versions work better with older Minecraft versions (doesn't refuse) than newer Minecraft versions do with older Java versions (always refuses if the minimum isn't being met). But the Eclipse Temurin Docker image by PufferPanel should first be updated in order to support Java 21. (I'd love to make some changes but it seems that the Docker image doesn't have a public source which you can contribute to, or well I saw the runtime images but I'm not completely sure about that and it are only a few small changes I think). Or does it use the original Temurin image? Since it doesn't specify a username. Specifying Java 21 already right now on the original system (non-Docker) (even though not default) completely works fine.

@Minionguyjpro Minionguyjpro changed the title Bump Default Java Version to 21 (Ensure Compatibility with 1.20.5+) [PUFFERPANEL'S TEMURIN DOCKER IMAGE NEEDS UPDATE TO SUPPORT JDK 21 FIRST] Bump Default Java Version Value to 21 (Ensure Compatibility with 1.20.5+) [PUFFERPANEL'S TEMURIN DOCKER IMAGE NEEDS UPDATE TO SUPPORT JDK 21 FIRST] May 1, 2024
@Minionguyjpro
Copy link
Contributor Author

Whoops, IT'S NOT DONE YET! I see that some things still need to be changed first!

@Minionguyjpro Minionguyjpro changed the title Bump Default Java Version Value to 21 (Ensure Compatibility with 1.20.5+) [PUFFERPANEL'S TEMURIN DOCKER IMAGE NEEDS UPDATE TO SUPPORT JDK 21 FIRST] Bump Minecraft Default Java Version Value to 21 (Ensure Compatibility with 1.20.5+) [PUFFERPANEL'S TEMURIN DOCKER IMAGE NEEDS UPDATE TO SUPPORT JDK 21 FIRST] May 1, 2024
@Minionguyjpro
Copy link
Contributor Author

Okay, everything should be fixed now. I made small changes to one mod so it's right again (since I checked and the names changed and the old versions weren't available anymore, so that's why). I also fixed that some things were containing '17' but should've not changed to '21' (I've used a sed pattern on Linux to change it from 17 to 21 for every file containing 'minecraft' in the name, it's great but it had also changed a few values that should've not been changed). I've fixed the incorrect parts as they should be, and now it should be good to go as soon as the template also gets a Java 21 version released.

@Minionguyjpro Minionguyjpro changed the title Bump Minecraft Default Java Version Value to 21 (Ensure Compatibility with 1.20.5+) [PUFFERPANEL'S TEMURIN DOCKER IMAGE NEEDS UPDATE TO SUPPORT JDK 21 FIRST] Bump Minecraft Default Java Version Value to 21 (Ensure Compatibility with 1.20.5+) [PUFFERPANEL'S TEMURIN DOCKER IMAGE NEEDS UPDATE TO SUPPORT JDK 21 FIRST, unless it uses the original Temurin image?] May 1, 2024
@LordRalex
Copy link
Member

https://github.com/pufferpanel/runtime-images is the correct repo that we use for the images that require "some" customization. Otherwise, using eclipse-temurim directly is okay.

I just bumped that to include 21 in it's pipeline.

@LordRalex LordRalex merged commit 250fb2d into pufferpanel:v2.6 May 1, 2024
1 check passed
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.

2 participants