Truinto / ONI-Modloader-SimpleMods

OxygenNotIncluded Mods
MIT License
16 stars 6 forks source link

[Linux] [Customize Buildings] Crashes ONI silently on startup #17

Closed Jemsterr closed 2 years ago

Jemsterr commented 2 years ago

Describe the bug A clear and concise description of what the bug is.

To Reproduce Steps to reproduce the behavior:

  1. Subscribe to Customize Bulidings
  2. Activate Customize Bulidings
  3. Load ONI on Linux
  4. Look at your destop as ONI silently disappears. [Edit: Klei logo appears, then crash]

[required] Debug Logs

Screenshots N/A

Desktop (please complete the following information):

Additional context This is the same error that I encountered in Fixed Ore Scrubber https://github.com/PsychoDad9999/ONI-Mods/issues/1

I don't know what he did to fix it, but linked here for what it's worth.

Jemsterr commented 2 years ago

Also see https://github.com/Truinto/Nightinggale-ONI-mods/issues/1 I just realised that that was your repo, too. Seems to be the same problem.

Truinto commented 2 years ago

Okay. That means that one (or more) of the hundreds patch signatures is illegal. Assuming that Klei changed a building, this could be any patch in the entire mod... The log file doesn't tell which and I have no way of testing it myself. Setting up a Linux machine isn't really an option for me.

I can only dig through every patch and check for that condition. That will take hours.

Maybe you can help me narrow the issue. Use this config file CustomizeBuildings.zip and try booting. If it works, enable options until it crashes. Most options have either a 'global'-flag or a enabled when they are not the default value (just at +1 to numbers).

Jemsterr commented 2 years ago

Thanks. I'll start the debug tonight after the wife and kids are in bed.

Jemsterr commented 2 years ago

Ran a diff between the two config files. Only adjusting the settings that had been changed between the .json that is shipped with the mod and the one you provided. The only setting that crashed the game was: "LadderCometInvincibility": true,

But I did not make these changes: image

Edit: For my final test, I took the original .json file, and changed the LadderCometInvincibility line to false. And no crash.

Truinto commented 2 years ago

Ugh, this feature again. For some reason that's already the 3rd time this one patch makes problems. Thanks a lot for the help. The fix took only a few minutes :)