-
Notifications
You must be signed in to change notification settings - Fork 87
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
Updated Bukkit to 1.20.6 #458
Updated Bukkit to 1.20.6 #458
Conversation
If anyone has any ideas as to why these dependencies aren't working that'd be really helpful:
|
If you're not aware it looks like a lot of work has already been done on this branch here https://github.com/PolyhedralDev/Terra/commits/dev/1.20.6/ |
Not really no, that's just the initial work required for 1.20.6. It's also only for Fabric not Paper. |
This is a pull request for that branch :) |
Hey! This happens because in the paper repo is marked as
But you're using
this is in buildSrc/src/main/kotlin/Versions.kt, anyways, you should use the newest devbundle with
|
Forget what i Said LMAO paperBuild and paperDevBundle are two separated repos and they're NOT synced in builds, so in order to use them both, you need to specify the versions for both! Use
and this
Anyways, this is actually weird. for some reason is asking for compiling in Java 17: |
Oooh ok, I'll give that a go! |
Sure! also, check the java thing lol |
Yeahh I'll try to work out what's going on there |
Not having much luck with this one unfortunately. |
weird, let me check your repo lol |
I'll commit my attempt real quick |
Sorry, i was busy, but i find what you need to change the repo in DependencyConfig.kt and in build.gradle.kts from https://papermc.io/repo/repository/maven-public/ to https://repo.papermc.io/repository/maven-public/ Also you need to change the Java version to 21 in CompilationConfig.kt but this breaks fabric builds so idk |
I did a bukkit build only with java version 21 and the Paper API error didn't happen but the NMS are totally broken, you also would need to migrate the bukkit API to the Bukkit long-standing API inside RegistryFetcher.java |
Ahhh ok, didn't know about the dependency kotlin file, I'll give that one a go. I'm not surprised that the NMS wasn't working, I've not been able to index so couldn't see any method names. |
I'm confused, as I think this was done in my initial commit Edit: It seems like the issee you were referring to was the issue in the Initializer interface. Resolving that at the moment |
I've tested this with the overworld pack on a 1.20.6 server, everything appears to be working as intended with no errors in console. |
builds fine, runs overworld 2.0 fine but i do get some errors and warnings: https://files.catbox.moe/wdw1m4.txt |
Is this error from another pack? Based off of the error I would assume you need to change |
its https://github.com/PolyhedralDev/TerraOverworldConfig/tree/2.0 and does not show up on 1.20.4 which is why i am mentioning this here |
I've just tested on the latest modrinth release on a 1.20.4 server and the issue is still present. |
no clue then, generated a 20kx20k world with chunky and didnt get it once |
Good spot, let me resolve this! |
That should be resolved in 9e60db4 |
I've updated this pr for Bukkit 1.21, I'm unsure if this reflection here This is currently untested |
Doing a test build now. Will try and setup a server to test on. |
failure.txt |
Have you builded the Spigot version only? this usually happens because you only ran gradle build instead of the specific platform build |
Whats the command for that again? |
Built your branch and paper 1.21, heres the log: |
Ahh, seems like cloud will need an update |
platforms/bukkit/common/src/main/java/com/dfsek/terra/bukkit/util/VersionUtil.java
Outdated
Show resolved
Hide resolved
There was one person who reported iit not even loading on spigot as of now |
Looks like there is an issue with Terra pulling version information on 1.21 Paper, have not tested Spigot. Log attached. |
Reverted 1.21 commits and moved them to #461 |
I really need your plugin for my new server :) Anything I can help with to get this running on paper for 1.20.6? |
It should run on 1.20.6 already if you compile it yourself |
These commits look good, tested this PR locally and all seems to be well. Thank you for your contributions! |
1.20.6 test jar as of 500cf3c Terra-bukkit-6.5.0-BETA+500cf3c0b-shaded.zip |
In this PR I have also set the Github gradle-build action to run on Java 21 as it is required by 1.20.6.
I am having some trouble getting the gradle dependencies to work so have been unable to test this at all.