architectury / architectury-plugin

A simple gradle plugin to enable developing multiplatform mods.
MIT License
56 stars 19 forks source link

Unable to disable Legacy Mixin AP with Architectury plugin enabled #41

Closed JT122406 closed 4 months ago

JT122406 commented 6 months ago

In loom 1.5 disabling Legacy AP became much larger, however arch plugin doesn't support this as it looks for refmap

JT122406 commented 6 months ago

This is seemly fixed in the last 2 commits however from my testing it still adds the refmap json for the mixins in common to the jar. This should not be the case as there should be no refmaps in the fabric jar at all if this option is disabled

shedaniel commented 6 months ago

Are you sure you are on the latest version and not just gradle caching the first version? It shouldn't add the refmap name in my testing.

JT122406 commented 6 months ago

Are you sure you are on the latest version and not just gradle caching the first version? It shouldn't add the refmap name in my testing.

Based on my testing this is with legacy ap disabled just in the fabric project image This is with it disabled in fabric and common: image However with this option the forge jar is missing the refmap: image