fabric-loader Conditionally apply mixins

Currently, to conditionally do something, you have virtually 2 choices. 1.

if (disabledFeature) return;

IMixinConfigPlugin. This is powerful, although slightly harder to use.

I think the mixin JSON could be extended to allow a condition to be passed next to the applicable mixin.

Feb 04 '19 19:02 Pannoniae

The format now allows potential appending of fields to the mixins list, as of 0.4.0.

Mar 05 '19 20:03 asiekierka

Can this be closed then?

Mar 05 '19 21:03 Pannoniae

No. You still need to use IMixinConfigPlugin.

Mar 05 '19 21:03 asiekierka

Can't we just always stick to mixin config plugins for conditional application? This is what sponge has been always doing. Need to take a stance on this and decide whether to kill this issue.

Nov 24 '19 05:11 liach

Just use IMixinConfigPlugin, it trivial to use and standard across Mixin environments - no need to introduce a Fabric-specific concept for this.

Jul 31 '20 11:07 jamierocks

We don't (officially) support mixin plugins, they don't work well with planned features like pre-baking bytecode edits and are too easy to mess up by loading classes that shouldn't be at this stage.

Jul 31 '20 11:07 Player3324