[1.12.2] Make CTM a proper client-side dependency

Question

[1.12.2] Make CTM a proper client-side dependency

Choonster opened this issue 7 years ago · 2 comments

Choonster commented 7 years ago

MinecraftForge/MinecraftForge#4403 added support for sided dependencies specifically for situations like Chisel and CTM, but Chisel still uses its old, slightly hacky code to detect whether CTM is installed on the client instead of declaring it as a client-side dependency.

It appears that @tterrag1098 intended to do that back in November 2017 (as mentioned here), but it never happened.

Drullkus · Answer 1 · 2018-03-18T09:05:55.000Z

@tterrag1098 The problem is older forge versions is that they will crash on that. It happened with Twilight Forest...

tterrag1098 · Answer 2 · 2018-03-18T09:37:51.000Z

I think it's been long enough. Plus our current way crashes a lot too.

…

On Mar 18, 2018 5:05 AM, "Drullkus" ***@***.***> wrote: @tterrag1098 <https://github.com/tterrag1098> The problem is older forge versions is that they will crash on that — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#661 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ADk-8D8ulYfYcPUpwC7gatiksO_bWjusks5tfiNzgaJpZM4SvBfa> .

Share to