-
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.21/1.21.1 #461
Updated Bukkit to 1.21/1.21.1 #461
Conversation
This is now functional - for anyone compiling this pr themselves please note that commands do not work currently. |
Thank you for your help. |
...forms/bukkit/nms/v1_21/src/main/java/com/dfsek/terra/bukkit/nms/v1_21/NMSInjectListener.java
Outdated
Show resolved
Hide resolved
1.21 test jar as of e7cc275 Terra-bukkit-6.5.0-BETA+e7cc27534-shaded.zip |
I've updated cloud to v2 - I haven't tested this quite yet and I am having some issues when reloading gradle (specifically regarding 'mixin-lifecycle' but I'm unsure if this was an previous issue or something I've caused. Edit: I tested this - getting some odd class not found exceptions, if someone else could try building that'd be great |
I was able to successfully build this just for bukkit. I didn't care to look through the stacktrace to see what went wrong, but I suspect it has to do with something fabric related. Testing the jar now, but if anyone wants to test it themsevles, it won't let me upload here so contact me on Discord: torrentstaff |
This is a build only for bukkit, I won't add fabric 1.21 support. If anyone is wanting to use a working build on 1.21 I'd use Aaron's build #461 (comment) until I've got Cloud v2 working |
If someone could test this version on 1.21.1 that'd be great as I have only done testing on 1.21 |
|
Make sure you're building my PR as the version mentioned in your error is the latest commit from the Terra main branch |
@ReferTV That is not the correct branch. You've built the master not the 1.21.1 version. |
@TobyWarren I can't build this version:
|
This PR is Bukkit only follow the instructions on the README to build a single platform |
@ReferTV "./gradlew :platforms:bukkit:build" |
Don't know if you still want feedback but i don't have any issue (so far) after upgrading to 1.21.1 with your PR. |
8d01059
to
dd272bc
Compare
This pr is waiting on the Cloud Framework to fix incompatibilities. I'll run some proper tests once paper's experimental builds are slightly less crashy.
If a
dev/1.21
branch could be made so I can redirect the pr towards that branch that would be great