PersonTheCat / OreStoneVariants

A powerful utility for generating new blocks when given a foreground and background.
GNU General Public License v3.0
7 stars 8 forks source link

Doesn't work on Server #141

Closed SHXRKIEgg closed 2 years ago

SHXRKIEgg commented 2 years ago

Trying to add it to my Better Minecraft Modpack and v7.0 and v7.1 doesn't work on Server

Issue - https://pastebin.com/6DUEx0gE

PersonTheCat commented 2 years ago

I can pretty much guarantee there is no client code leak on the server side. However, I am aware of a couple platform compatibility issues that only affect the server side. There is an unpublished fix for compatibility with Sponge-based platforms and the mod simply does not work with Java 11+.

Guys, I have to make this clear. Please do not share a partial crash report with no other details. I need the whole entire crash report, as well as diagnostics about your setup and what changed that caused this problem. And, if you really want this bug fixed soon, please consider reproducing the crash with fewer than 200 mods. Next time, I will close the ticket with no comment and just mark it as insufficient.

I need

SHXRKIEgg commented 2 years ago

Just tested with just the latest version of OSV and CatLib and still a mod loading error.

SHXRKIEgg commented 2 years ago

crash-2021-12-29_23.52.49-fml.txt Never mind the crash is here

PersonTheCat commented 2 years ago

Okay, I gotcha. This is just a warning message, but it throws exceptions for all errors on the server side. It's just trying to make a backup, but the file isn't there yet (which can be ignored on the client side, but not for server admins).

This is technically not a bug, but it is un-ergonomic. I'll just update the default behavior from CatLib in a day or so. In the meantime, just run the server from the client side and then copy all your configs over. Let me know if you try that and it does or does not work. I'll also update here when the new CatLib version is out.

SHXRKIEgg commented 2 years ago

Yeah seems to work when I use the Configs from the client side thanks.

PersonTheCat commented 2 years ago

Thanks! I'll update CatLib to only throw exceptions based on your configured errorLevel in the CatLib config file. While I'm here, I can also take this opportunity to have it display multiple errors at a time on the server side like it does on the client side. Will work on that today.

PersonTheCat commented 2 years ago

Hey, I've released an update to OSV to provide compatibility with Sponge servers, as well as an update to CatLib to update the warning behavior so it doesn't crash out of the box. Let me know if you try it and everything works for you. Will close this ticket later tomorrow, otherwise.