Load stage error Minecraft 1.19.2 S&R 1.4.2 Forge 43.2.14
SleeplessF0X opened this issue · 16 comments
Description
Game stopped on loading stage (window didn't opened) because of the conflict with SecurityCraft 1.9.6.1
Reproduction
Launch the game, wait ~30 seconds, close the Java Minecraft process
Logs
Software Info
- OS: Linux Mint 21
- Minecraft version: 1.19.2
- Minecraft launcher: Legacy Launcher
- Forge version: 43.2.14
- S&R version: 1.4.2
Additional info
I've got a feeling that the error is more on the SecurityCraft side, but since I don't know much about mod development I've came here first because the error came up to me after I tried to update this mod on my private server. If this issue needs to be reported to the SecurityCraft devs, let me know. Also, even though I didn't worked on any mods, I am a professional programmer, so if someone can point me to how to fix this - I'll be happy to work on contributing to the fix maybe even today
This is a fundamentally tricky issue, since only one mod can redirect a certain method at a time, and it is a very similar redirector to SC's. A possible solution would be to increase the priority (since mixins follow natural ordering and priority 1 beats priority 2), make the redirect optional, and add a mixin into one of securitycraft's methods
Oh, I see, thanks for the explanation! So it requires you to tune the mixin's priority and for SC to implement a compatibility with your mod?
Yep. That's precisely it. What IThundxr and I have done is made it so that our mixins load after SecurityCraft's, in the process replacing a few of SC's redirects. We patched our redirects to adopt the functionality of SC's where needed, so it should be fully compatible once 1.4.3 releases.
about to release, if you can confirm it works for you once it does, I can close this issue
I will be able to test somewhere in an hour, so you can wait for information around this time
I am having the same issue. I will be able to test as well when the patch is released.
I've ran the release workflows now all we can do is wait for GitHub to build and publish the mod
Oh dear. Looks like the changes broke more things.
Oh dear. Looks like the changes broke more things.
Disable datapack_reload_exceptions in modernfixes config
Oh dear. Looks like the changes broke more things.
crash-2023-07-07_09.56.08-fml.txtDisable datapack_reload_exceptions in modernfixes config
Seems to have fixed it, I'm launching fine now. Little strange though how disabling that fixes the issue if it just hides erroring recipes and the like.
Thanks for your time mate.
Everything works fine for me too (after disabling datapack_reload_exceptions), played around a bit and didn't see any other bugs or errors. Thanks for your time, you've got golden hands, people
I'm a little curious why disabling that fixes the problem though. Ideally I would like it on
I'm a little curious why disabling that fixes the problem though. Ideally I would like it on
It's a modern fix thing that's been fixed but it's only in dev versions atm the modern fix dev told me that's they'll release a working (fixed) version soon and which should make changing that config not needed