PolyhedralDev / Terra

Voxel world generation modding platform
MIT License
642 stars 85 forks source link

Port to Folia #398

Closed pontaoski closed 11 months ago

pontaoski commented 1 year ago

Pull Request

Brief description.

This adds Folia compatibility.

What Issues Does This Fix?

Fix #397

Licensing

Goal of the PR

Affects of the PR

Types of changes

Compatiblity

Contribution Guidelines.

pontaoski commented 1 year ago

grafik seems to be working? :3

solonovamax commented 1 year ago

should the folia port be a separate submodule/platform?

and, why is the 1.19_r1 nms not being shaded?

pontaoski commented 1 year ago

folia is 99% the same code, so the usual way to port is to just switch the API package to folia and guard folia APIs behind version checks

also oops that was failing to build locally so i commented it out

solonovamax commented 1 year ago

I opened an issue on run-task (jpenilla/run-task#31) requesting a plugin be added with folia support.

The reason I say it might warrant being a separate module is because it would allow as to remove the version checks, and then also being able to run a folia server to test during dev.

if it were a separate module, it would be best to somehow commonize the paper module so that the paper & folia modules have very few lines of code.

solonovamax commented 1 year ago

nvm, run-paper already has folia support. run-paper should be updated to the latest version, 1.1.0

solonovamax commented 1 year ago

we should attempt to have 1.19_r1 shaded again

solonovamax commented 1 year ago

also, idk about setting the folia dep to 1.20. that means that we could accidentally introduce changes that would prevent it from running on 1.18 bukkit.

pontaoski commented 1 year ago

folia doesn't come in a 1.18 API flavour

solonovamax commented 1 year ago

yes I know. which is another reason for having both a folia and a bukkit platform.

justaureus commented 12 months ago

also, idk about setting the folia dep to 1.20. that means that we could accidentally introduce changes that would prevent it from running on 1.18 bukkit.

We don't support 1.18 bukkit anymore, only 1.19/1.20.

solonovamax commented 12 months ago

We don't support 1.18 bukkit anymore, only 1.19/1.20.

ah, I see

however, it'd still (possibly) break on 1.19

duplexsystem commented 11 months ago

Superseded by #417