TalonFloof / EnhancedWeather

Adds Localized Weather Systems to Minecraft
https://modrinth.com/mod/enhanced-weather
MIT License
21 stars 3 forks source link

Server crash with latest version of mod & its dependencies installed #14

Closed Arriochh closed 1 year ago

Arriochh commented 1 year ago

When I upload Fabric API (version as instructed), Cloth Config (latest version for 1.19.2), and enhanced weather (latest version), the server fails to start. I have only these three in the mods folder. I reckon this is not compatible with dedicated servers for the time being, but I just wanted to file a report in case.

Unfortunately, the server does not treat this as a crash, so I am only able to pull logs instead of crash reports. The server tries to restart the server after it "crashes" in which a report is unable to be filed.


LOGS

[20:24:50] [main/INFO]: Loading Minecraft 1.19.2 with Fabric Loader 0.14.10 [20:24:50] [main/INFO]: Loading 44 mods:

TalonFloof commented 1 year ago

Can confirm this issue, it appears the MixinWorld class is causing this error. I'll try to release a patch to fix this problem.

Here's the error message when using my development environment.

org.spongepowered.asm.mixin.transformer.throwables.MixinPreProcessorException: Attach error for enhancedweather.mixins.json:MixinWorld from mod enhancedweather during activity: [Transform -> Method getRainGradient(F)F -> INVOKESTATIC -> net/minecraft/client/MinecraftClient::getInstance:()Lnet/minecraft/client/MinecraftClient;]

TalonFloof commented 1 year ago

Update, Looks like this might be harder than I thought. I fixed the issue with the MixinWorld class. But it looks like Cloth Config breaks when using AutoConfig with dedicated servers, so I'm going to have to rewrite my config system.

This is the error I got java.lang.NullPointerException: Cannot invoke "me.shedaniel.autoconfig.ConfigData.validatePostLoad()" because "this.config" is null

The following can also be found on the wiki.

Please DO NOT use Auto Config / Cloth Config for server mods

TalonFloof commented 1 year ago

@Arriochh Fixed this issue. Will release a new version with the bug fix today.