-
Notifications
You must be signed in to change notification settings - Fork 16
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
Crash on Quilt caused by terraform-wood-api #71
Comments
That is a Quilt API, so switching to use that API would not be Fabric compatible... |
There is Quilted Fabric API, but need additional work to make it compatible with Quilt as well |
I often see claims Fabric mods are compatible with Quilt as long as they aren't using Fabric internals, which we aren't doing here, so I consider this kind of result to be a bug in Quilt (in QFAPI specifically). I don't have the requisite combination of time and motivation at the moment to either produce a Quilt-specific version of Terraform API or add logic to Terraform API to register things differently depending on which loader is active. If somebody else wants to contribute work on a Quilt release, I believe Terraformers would be happy to accept it, subject to code quality review. |
Note, according to TerraformersMC/Traverse#95 (comment) This will be fixed by QuiltMC/quilt-standard-libraries#253 |
I haven't heard any more about this in a while and hopefully the Quilt fix means everybody is happy. |
Yes indeed, it is working fine for some time now 😊 Thanks for looking into this anyway while not being an issue on your side 👍 |
Hi there,
I tried to load
traverse-6.1.2
(for MC 1.19.3) via Quilt Modloader0.18.1-beta25
(withqfapi-5.0.0-alpha.6_qsl-4.0.0-beta.2_fapi-0.69.1
). Crashes with these errors/logs:Does not happen on Fabric Loader (
0.14.11
withfabric-api-0.69.1
).Any idea what could be the reason (and fix)? I already looked at net.minecraft.entity.data.TrackedDataHandlerRegistry, but there is no
set()
function (as the error message is suggesting to use).The text was updated successfully, but these errors were encountered: