Slimefun4 icon indicating copy to clipboard operation
Slimefun4 copied to clipboard

Spigot/CB limits Java version...

Open WalshyDev opened this issue 2 years ago • 3 comments

📍 Description

Why they do this is beyond me but it means our supported versions aren't actually the same (At least for Spigot/CB, Paper removed this code).

1.14.4 capped at Java 12: https://hub.spigotmc.org/stash/projects/SPIGOT/repos/craftbukkit/browse/src/main/java/org/bukkit/craftbukkit/Main.java?until=e73aabd66662dd805cc1d9d992b6f7eb3513b8ab#150-154

1.15.2 capped at Java 13/14: https://hub.spigotmc.org/stash/projects/SPIGOT/repos/craftbukkit/browse/src/main/java/org/bukkit/craftbukkit/Main.java?until=6881a108035ee461f43c39300c5bf67c49598ce7#151-155

1.16.5 capped at Java 15/16: https://hub.spigotmc.org/stash/projects/SPIGOT/repos/craftbukkit/browse/src/main/java/org/bukkit/craftbukkit/Main.java?until=f3f30947e07c4ea07dcbf41129c2ca830a73fd4d#151-154

1.17.2 capped at Java 16/17: https://hub.spigotmc.org/stash/projects/SPIGOT/repos/craftbukkit/browse/src/main/java/org/bukkit/craftbukkit/Main.java?until=fb556bfa1404661b38cc3abd1f0a7d02764f8f5b#157-159

1.18.2 capped at Java 17/18: https://hub.spigotmc.org/stash/projects/SPIGOT/repos/craftbukkit/browse/src/main/java/org/bukkit/craftbukkit/Main.java?until=05a38da190704cb8df90401c45fc39d087cdc031#156-159

1.19 capped at Java 18: https://hub.spigotmc.org/stash/projects/SPIGOT/repos/craftbukkit/browse/src/main/java/org/bukkit/craftbukkit/Main.java?until=0231a37464e6455bb6ab21ea8c7ca560c354147f&untilPath=src%2Fmain%2Fjava%2Forg%2Fbukkit%2Fcraftbukkit%2FMain.java#156-159

(The secondary versions are ones added usually a few months later)

Per this, our Java 16 work has limited the version support to 1.16.5 (later versions) or 1.17.x.

Paper removes this version check: https://github.com/PaperMC/Paper/blob/d0e7fa4dc95adf25d01736194a389e768250537f/patches/server/0362-Remove-garbage-Java-version-check.patch

I do not know what version this was added in.

💻 Server Software

Spigot

🎮 Minecraft Version

1.14.x - 1.16.5(ish)


I am making this as a note for us to discuss what we want to have some comments and refer for any changes.

WalshyDev avatar Jul 24 '22 18:07 WalshyDev

1.15.2 caps at Java 14, didnt test 1.14.4

ybw0014 avatar Jul 24 '22 18:07 ybw0014

We can drop support for 1.14/1.15 eventually anyway. Noone uses it.

The fact that no issue about it has been reported thus far shows that too 👀

TheBusyBiscuit avatar Jul 24 '22 19:07 TheBusyBiscuit

I would say drop support since close to 0 servers are using versions before 1.16 and it would allow Slimefun to use the new items added

char3210 avatar Jul 31 '22 16:07 char3210