Skip to content
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

Closed
koelle25 opened this issue Dec 20, 2022 · 7 comments
Closed

Crash on Quilt caused by terraform-wood-api #71

koelle25 opened this issue Dec 20, 2022 · 7 comments

Comments

@koelle25
Copy link

Hi there,

I tried to load traverse-6.1.2 (for MC 1.19.3) via Quilt Modloader 0.18.1-beta25 (with qfapi-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 with fabric-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).

@gniftygnome
Copy link
Contributor

That is a Quilt API, so switching to use that API would not be Fabric compatible...

@RoonMoonlight
Copy link

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

@gniftygnome
Copy link
Contributor

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.

@gniftygnome
Copy link
Contributor

Note, according to TerraformersMC/Traverse#95 (comment)

This will be fixed by QuiltMC/quilt-standard-libraries#253

@gniftygnome
Copy link
Contributor

I haven't heard any more about this in a while and hopefully the Quilt fix means everybody is happy.

@koelle25
Copy link
Author

koelle25 commented Jun 6, 2023

Yes indeed, it is working fine for some time now 😊 Thanks for looking into this anyway while not being an issue on your side 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants