From b90ce217aba50718f6f054853c8269e166d7cad8 Mon Sep 17 00:00:00 2001 From: modmuss50 Date: Wed, 12 May 2021 20:29:23 +0100 Subject: [PATCH] Improve merge jars error handling --- .../loom/configuration/providers/MinecraftProvider.java | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/src/main/java/net/fabricmc/loom/configuration/providers/MinecraftProvider.java b/src/main/java/net/fabricmc/loom/configuration/providers/MinecraftProvider.java index 89c1646a..9416ff6d 100644 --- a/src/main/java/net/fabricmc/loom/configuration/providers/MinecraftProvider.java +++ b/src/main/java/net/fabricmc/loom/configuration/providers/MinecraftProvider.java @@ -31,7 +31,6 @@ import java.net.URL; import java.nio.charset.StandardCharsets; import java.util.Optional; import java.util.function.Consumer; -import java.util.zip.ZipError; import com.google.common.io.Files; import com.google.gson.GsonBuilder; @@ -101,11 +100,11 @@ public class MinecraftProvider extends DependencyProvider { if (!minecraftMergedJar.exists() || isRefreshDeps()) { try { mergeJars(getProject().getLogger()); - } catch (ZipError e) { + } catch (Throwable e) { deleteFiles(); getProject().getLogger().error("Could not merge JARs! Deleting source JARs - please re-run the command and move on.", e); - throw new RuntimeException(); + throw e; } } }