- Sponsor
-
Notifications
You must be signed in to change notification settings - Fork 133
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
XNet (filtered) mekanism gas channel server crash #1170
Comments
Debugged this with @stefancdr. This issue caused our server to crash when updating from 1.62 to 1.63(a). In case any other server admin bumps into this; simply change the forge.cfg file to set the "removeErroringTileEntity" flag to true. This will remove the issue-throwing Xnet connectors and get your server started again. Please do note that this might influence other stuff as well if you have any other tile entities that are being a dick. So be sure to have a backup of your server! |
Just found the correct repo for Ynot @ https://git.asie.pl/asie-minecraft/YNot I might look into this tomorrow and see if we can get a fix in place and PR'ed
|
Pull request made (asiekierka/YNot#13), let's hope I didn't make any silly coding mistakes. (this version works locally) |
Great work! |
0.2.4 is now available :-). |
Modpack version
1.63a
Can the issue be reproduced?
Yes
Logs
https://pastebin.com/djMBU20Z (server log, SMP)
https://pastebin.com/ny9T50gz (server log, SP)
Issue
When using mekanism gas channels, the server crashes as soon as gas is moved (the entity is ticked). This is most likely caused by the update to the new backwards-compatibility breaking versions of McJty's mods; YNot hasn't seen an update in a year (according to GitHub) and so probably isn't compatible.
Xnet's commit that broke this is this: McJtyMods/XNet@bad2273#diff-ccdebad88af10ecb4fad7ab360007b5e
YNot should be updated to link to mcjty.xnet.config.ConfigSetup instead of GeneralConfiguration, but I can't seem to find the source code for 0.2.3, only 0.2.2 is available on GitHub, otherwise I'd fork it and fix it myself :-).
The text was updated successfully, but these errors were encountered: