alcatrazEscapee / Chocolate

A mod that fixes some annoying bugs with vanilla's data driven world gen in 1.16
https://www.curseforge.com/minecraft/mc-mods/chocolate-fix
MIT License
4 stars 3 forks source link

Chocolate has unlisted dependency on Forge 34.1.33, resulting in unexpected crashes with anything lower #2

Closed BedrockLegends closed 3 years ago

BedrockLegends commented 3 years ago

crash-2020-10-29_22.19.21-client.txt

alcatrazEscapee commented 3 years ago

I'm not sure how you arrived at that conclusion. The crash log is saying that an undergarden:stoneborn entity is somehow causing an explosion, which is crashing at Server Utils com.github.elrol.ElrolsUtilities.events.BlockEventHandler.

Unfortunately, I cannot find any reference to this class in Server Utilities source code to confirm my suspicions. I highly recommend you report the issue to either Server Utilities, or The Undergarden as they seem to be involved. I have asked the developers of the former about this to see if they have anything to add.

Any more info would be appreciated as I'm pretty sure this is not Chocolate's bug.

alcatrazEscapee commented 3 years ago

msg

After asking the author, this sounds like it's probably an issue with Server Utilities and/or The Undergarden and should be reported as such. I'm closing this as I see nothing that Chocolate is doing to cause this, nor anything I would be able to do to fix this.

Elrol commented 3 years ago

The issue isnt my mod. I am just adding a few checks to make sure that mods that dont implement things properly dont crash the server.

One of my more active testers has over 200 mods and it was working fine. He reported the issues after adding your mod. So something must not be playing well together. Either way i will attempt to fix the issues introduced by mod conflicts.

alcatrazEscapee commented 3 years ago

If that is the case, I'd like to see anything in there that would point to it being Chocolate's error. I fix two biome serialization bugs in vanilla. Both of which, if they throw exceptions (which they have been written to do in error states), will clearly identify that Chocolate has thrown them. i.e. here.

However, looking at that crashlog

Is there an actual issue here and the wrong crash log was uploaded perhaps?

BedrockLegends commented 3 years ago

crash-2020-10-30_18.55.53-client.txt

latest.log

debug.log

As soon as I put this Mod into my pack, it crashes even on the new update, If I dont add this mod at all then nothing crashes. This is all on your mod as it only happens after adding it

alcatrazEscapee commented 3 years ago

That debug log is very interesting. A couple things that stand out:

[30Oct2020 19:18:22.881] [main/WARN] [mixin/]: Error loading class: net/minecraftforge/registries/ForgeRegistryEntry$UncheckedRegistryEntry (java.lang.ClassNotFoundException: null)

This is due to the fact Chocolate was built against Forge 34.1.33, which introduced UncheckedRegistryEntry. However I have made a mistake and not set the required Forge version in the mods.toml to this version. As per the log, this is currently using Forge 34.1.25. So there is issue no. 1.

java.lang.NoClassDefFoundError: Could not initialize class net.minecraft.util.registry.Registry
    at net.minecraft.item.ItemStack.lambda$static$3(ItemStack.java:77) ~[?:?] {re:mixin,re:classloading,xf:fml:apotheosis:apothitemusehook,pl:mixin:APP:forbidden_arcanus.mixins.json:ItemStackMixin,pl:mixin:APP:quark.mixins.json:ItemStackMixin,pl:mixin:A}

This is also highly concerning. This would seem to indicate that a transformer has ran into a conflict, although nothing here points to Chocolate being the cause. (as this is transforming ItemStack, something Chocolate does not go anywhere near).

So: update your forge version to 34.1.33. I will make chocolate require it as of the next build. If this still crashes, then post the debug.log again and we can investigate, capiche?

BedrockLegends commented 3 years ago

@alcatrazEscapee I cant update my forge version as Mods are still crashing in my pack cause of the new forge breaking changes

alcatrazEscapee commented 3 years ago

grumbles incoherently

Well then I'll see what I can do.

BedrockLegends commented 3 years ago

Well if this helps so far out of 70 mods + yours Immersive Engineering crashes

alcatrazEscapee commented 3 years ago

Okay, the process of backporting through Forge versions begins...

First step: Remove the unnecessary reference to UncheckedRegistryEntry. It now compiles against 34.0.0. Fantastic! Except now it crashes on anything lower than 34.1.31, because guess what - this mod relies quite a lot on a fix added in that version (one that I had made to Forge).

So there may be more bodging required...

Although in the meantime, identifying which mods are broken on newer versions of Forge and reporting it to them so they can update will always be helpful :)