From dcbe6c456657d47a277e2e4715fc017fbd1cb2ba Mon Sep 17 00:00:00 2001 From: modmuss50 Date: Wed, 11 Dec 2019 14:19:47 +0000 Subject: [PATCH] Possible fix/workaround for idea enabling annotation processors when we dont really need them. --- src/main/java/net/fabricmc/loom/LoomGradleExtension.java | 4 ++++ .../java/net/fabricmc/loom/util/LoomDependencyManager.java | 6 +++++- .../net/fabricmc/loom/util/SetupIntelijRunConfigs.java | 7 ++++--- 3 files changed, 13 insertions(+), 4 deletions(-) diff --git a/src/main/java/net/fabricmc/loom/LoomGradleExtension.java b/src/main/java/net/fabricmc/loom/LoomGradleExtension.java index 959f573..7a05416 100644 --- a/src/main/java/net/fabricmc/loom/LoomGradleExtension.java +++ b/src/main/java/net/fabricmc/loom/LoomGradleExtension.java @@ -290,4 +290,8 @@ public class LoomGradleExtension { return refmapName; } + + public boolean ideSync() { + return Boolean.parseBoolean(System.getProperty("idea.sync.active", "false")); + } } diff --git a/src/main/java/net/fabricmc/loom/util/LoomDependencyManager.java b/src/main/java/net/fabricmc/loom/util/LoomDependencyManager.java index fce9a26..4893573 100644 --- a/src/main/java/net/fabricmc/loom/util/LoomDependencyManager.java +++ b/src/main/java/net/fabricmc/loom/util/LoomDependencyManager.java @@ -156,6 +156,7 @@ public class LoomDependencyManager { } private static void handleInstallerJson(JsonObject jsonObject, Project project) { + LoomGradleExtension extension = project.getExtensions().getByType(LoomGradleExtension.class); JsonObject libraries = jsonObject.get("libraries").getAsJsonObject(); Configuration mcDepsConfig = project.getConfigurations().getByName(Constants.MINECRAFT_DEPENDENCIES); Configuration apDepsConfig = project.getConfigurations().getByName("annotationProcessor"); @@ -166,7 +167,10 @@ public class LoomDependencyManager { ExternalModuleDependency modDep = (ExternalModuleDependency) project.getDependencies().create(name); modDep.setTransitive(false); mcDepsConfig.getDependencies().add(modDep); - apDepsConfig.getDependencies().add(modDep); + + if (!extension.ideSync()) { + apDepsConfig.getDependencies().add(modDep); + } project.getLogger().debug("Loom adding " + name + " from installer JSON"); diff --git a/src/main/java/net/fabricmc/loom/util/SetupIntelijRunConfigs.java b/src/main/java/net/fabricmc/loom/util/SetupIntelijRunConfigs.java index 018eb0c..5066746 100644 --- a/src/main/java/net/fabricmc/loom/util/SetupIntelijRunConfigs.java +++ b/src/main/java/net/fabricmc/loom/util/SetupIntelijRunConfigs.java @@ -59,9 +59,10 @@ public class SetupIntelijRunConfigs { } private static void generate(Project project) throws IOException { - //Ensures the assets are downloaded when idea is syncing a project - if (Boolean.parseBoolean(System.getProperty("idea.sync.active", "false"))) { - LoomGradleExtension extension = project.getExtensions().getByType(LoomGradleExtension.class); + LoomGradleExtension extension = project.getExtensions().getByType(LoomGradleExtension.class); + + if (extension.ideSync()) { + //Ensures the assets are downloaded when idea is syncing a project MinecraftAssetsProvider.provide(extension.getMinecraftProvider(), project); MinecraftNativesProvider.provide(extension.getMinecraftProvider(), project); }