Obliterate phonos - #703
The-Shortman wants to merge 3 commits into
Conversation
All the other changes ensure that it won't break progression or worlds Reason being it has a server-crashing bug for which the developers refuse to add a one-line check to fix the crash I understand version chasing but it boils my blood when developers don't provide the most basic of bugfixes for older versions
|
After some further digging the situation is as follows:
It is entirely possible to fork the 1.18 branch of Phonos and maintain it ourselves as a separate project, without removing it from the modpack. In that case, this pull request becomes redundant and can be closed without merging. It has been suggested that we could patch it via a mixin in Astral Additions, but I think it would be more beneficial to do the fork, as it means the three Fabric 1.18.2 players outside of Astral may benefit from a functional mod on more recent Fabric versions than 0.14. |
|
I have placed this as "Won't Do" for now, as we are planning a major overhaul of our development team structure, and we have mitigations for this crash in place coming in 2.1.5 anyway. We can revisit this for 2.1.6 after we're more organised. |
ethanicusss
left a comment
There was a problem hiding this comment.
Our phonos structure replacements should be deleted too, but nothing will be broken if they remain
|
Ah true, then the better option will probably be to apply the patched version of the mod, since that shouldn't cause those problems. I do like the mod, and having to remove the radio jukebox was a shame, so having the ability for it be brought back in 2.1.6 would be excellent. |
|
This pull request is incompatible with #770. |
What does this pull request do?
Removes Phonos from the modpack, as it caused a game-crashing issue, with a very simple one-line patch, however the developers actively refuse to apply the patch and suggest downgrading the Fabric version instead.
Changes made
Additional Information
Potential Compatibility Issues