search

FabricMC / fabric-mixin-compile-extensions

Extensions for Mixin's annotation processor.
MIT License
8 stars 8 forks source link

Use mixin-processor as dependency in order to let loom pull the correct dependency for annotation processor. #14

Open LogicFan opened 3 years ago

LogicFan commented 3 years ago

Currently the extension use mixin as a dependency, hence it requires many additional jar to mess up with the annotationProcessor dependency added by loom when build mod. I think the correct way should change this from net.fabricmc:sponge-mixin:0.9.1+mixin.0.8.2 to net.fabricmc:sponge-mixin:0.9.1+mixin.0.8.2:processor and in the loom side remove additional dependencies added to the annotationProcessor.

https://github.com/FabricMC/fabric-mixin-compile-extensions/blob/ead78abfcbbaa2569c8b2748a855cc91fa192f89/build.gradle#L31-L35

This relates to https://github.com/FabricMC/fabric-loom/pull/441 & https://github.com/FabricMC/fabric-loader/issues/480.