Disable useLegacyMixinAp check in getDefaultRefmapName()

Fixes #242.
architectury · Dec 7, 2024 · eb593da · eb593da
1 parent aafd69b
commit eb593da
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/src/main/java/net/fabricmc/loom/extension/MixinExtensionImpl.java b/src/main/java/net/fabricmc/loom/extension/MixinExtensionImpl.java
@@ -61,7 +61,10 @@ public MixinExtensionImpl(Project project) {
 
 	@Override
 	public Property<String> getDefaultRefmapName() {
-		if (!super.getUseLegacyMixinAp().get()) throw new IllegalStateException("You need to set useLegacyMixinAp = true to configure Mixin annotation processor.");
+		// Arch: We need to access afterEvaluate state in useLegacyMixinAp's convention, so let's not query it.
+		// Otherwise, this property can't be set in a buildscript without afterEvaluate.
+		// https://github.com/architectury/architectury-loom/issues/242
+		// if (!super.getUseLegacyMixinAp().get()) throw new IllegalStateException("You need to set useLegacyMixinAp = true to configure Mixin annotation processor.");
 
 		return defaultRefmapName;
 	}