Skip to content
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

[1.12.2] Crash on newer Forge #7

Open
Diamondman121314 opened this issue Apr 8, 2020 · 11 comments
Open

[1.12.2] Crash on newer Forge #7

Diamondman121314 opened this issue Apr 8, 2020 · 11 comments

Comments

@Diamondman121314
Copy link

Diamondman121314 commented Apr 8, 2020

NEC crashes on the newer 1.12.2 Forge versions, which would be 2852+.

https://gist.github.com/Diamondman121314/8ce028767f32f74bac4da69b5493bc5a

@Diamondman121314 Diamondman121314 changed the title [1.12.2] Crash on latest Forge [1.12.2] Crash on newer Forge Apr 14, 2020
@Dakedres
Copy link

please fix tbh

@ghost
Copy link

ghost commented Jun 3, 2020

(a typical request for a fix)

@DDX5-DX12-Media
Copy link

Same Happens To Me. I Think It May Be The Sound Lib Playing Havoc Here, Not The Mod's Fault.

@boq
Copy link
Member

boq commented Jul 26, 2020

It's Forge issue, caused by removal of exclude "paulscode/**" from signing task: MinecraftForge/MinecraftForge@db6ab2a#diff-c197962302397baf3a4cc36463dce5eaL246

@DDX5-DX12-Media
Copy link

@boq Do You Know The Paulscode Version Used

@boq
Copy link
Member

boq commented Jul 26, 2020

Paulscode version is not a matter here, it's Forge signing one of their classes (https://github.com/MinecraftForge/MinecraftForge/blob/1.12.x/src/main/java/paulscode/sound/codecs/CodecIBXM.java)

@DDX5-DX12-Media
Copy link

@boq I see That, Have You Tried To Add A Dependency For The Paulscode Lib So It Overrides The Forge Embedded Lib Instead

@DDX5-DX12-Media
Copy link

My Bad, Forgot To Elaborate. Open The Build.gradle file And Add The Following.
compile group: 'com.github.nifty-gui', name: 'nifty-pauls-soundsystem', version: '1.4.2'

@boq
Copy link
Member

boq commented Jul 26, 2020

That won't fix anything, as mod classes are loaded after Forge. Also compile does not mean it will be available during loading in modding setup, as everything needs to be inside single jar file.

Solutions are either to repackage CodecWav (not doing that due to license problems) or remove WAV support (and I don't want to do narrow supported formats), so I don't see that being resolved on my side any time soon.

@DDX5-DX12-Media
Copy link

I See, That's A Bit Of A Shame That They Had To Do That.

@ChromaPIE
Copy link

ChromaPIE commented Jan 16, 2022

In case anyone missed it, there's a fork available for newer forge: https://www.curseforge.com/minecraft/mc-mods/justenoughcodec

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants