Fabric API

Fabric API

152M Downloads

Fabric mixin conflict with EBE mod

XraveMC opened this issue ยท 8 comments

commented

1.21.4 fabric 0.16.14
Fabric api v0.119.3
Enhanced block entities v0.11.3

Log:
Method overwrite conflict for getModel in fabric-model-loading-api-v1.mixins.json:BakedModelManagerMixin from mod fabric-model-loading-api-v1, previously written by foundationgames.enhancedblockentities.mixin.BakedModelManagerMixin. Skipping method

commented

the mixin in question doesn't seem to be present in the source or the jar for the latest version of ebe. Feel free to object if I got it wrong, but I'll close this for now

commented

Okay. I'll investigate again in a minute and reopen if necessary

commented

the mixin in question doesn't seem to be present in the source or the jar for the latest version of ebe. Feel free to object if I got it wrong, but I'll close this for now

Ah my bad, I said the wrong version
1.20.1
Fabric api v last
EBE v last

commented

So this is actually a conflict in 1.20.1, but we expose an API to do exactly what their mixin is doing, and they already depend on fabric API, so the author, or someone else maintaining the mod should update the fabric API dependency, and use the supported API rather than a mixin. I imagine when that version was compiled, a supported API didn't exist. This isn't the fault of the ebe devs, but fabric isn't going to fix this as it's a breaking API change, whereas on their end it's just a small internal tweak to use the fabric method

commented

It's also possible for another mod dev to write a near single file hotfix to make it compatible if the ebe devs aren't willing. Feel free to find me on the fabric discord after talking with them

commented

It's also possible for another mod dev to write a near single file hotfix to make it compatible if the ebe devs aren't willing. Feel free to find me on the fabric discord after talking with them

So I report it to Dev EBE?

commented

That's what I would try first.

commented

Probably link this issue for context too.