PacifistMC / Forgix

A Gradle plugin/an Architectury addon to merge mod-loaders into one jar! 𝘸𝘒𝘩𝘩 𝘡𝘦𝘀𝘩𝘯𝘰𝘭𝘰𝘨𝘺

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Merge JAR process doesn't update mixin refmap JSON files?

Kitteh6660 opened this issue Β· comments

For some reason, if I merge JAR files, and when I compare Mixin refmaps, I've noticed that it does not automatically update the refmap if I use additional relocate. That means if the mod has Mixins, it will crash.

Particularly, this would happen on Fabric and Forge because they need to have refmap that is bundled with JAR file.

I'm using Forgix 1.2.9 if that helps.

commented

What are you using additionalRelocate for? It was meant to relocate libraries

I am using additionalRelocate for relocating Forge and Fabric-side code into their corresponding folders and it just so happens that the mixins are within those folders.

commented

@Kitteh6660
It's actually intentional that Forgix doesn't touch mixins as things can break horribly if it did
There may be other ways of achieving what you're trying to do

Just to be specific, the refmap JSON file. The JSON file below was manually done by hand because otherwise, Forge mixin refmap will try to refer to the wrong location and thus crashes the game. As a result, updating Mixin class files can be a bit tedious process. Which thankfully, am planning to ditch Forge entirely in 1.21.

dramaticdoors_forge.refmap.json