From 5f16dbb61a32a73775a74da813a53ed757a64074 Mon Sep 17 00:00:00 2001
From: Rhys <98863820+rhysdh540@users.noreply.github.com>
Date: Sat, 16 Nov 2024 17:07:48 -0500
Subject: [PATCH] don't add remapSourcesJar by default

---
 .../unimined/api/minecraft/MinecraftConfig.kt |  6 +++-
 .../internal/minecraft/MinecraftProvider.kt   | 28 +++++++++----------
 2 files changed, 18 insertions(+), 16 deletions(-)

diff --git a/src/api/kotlin/xyz/wagyourtail/unimined/api/minecraft/MinecraftConfig.kt b/src/api/kotlin/xyz/wagyourtail/unimined/api/minecraft/MinecraftConfig.kt
index 5e7d5d83..074f9405 100644
--- a/src/api/kotlin/xyz/wagyourtail/unimined/api/minecraft/MinecraftConfig.kt
+++ b/src/api/kotlin/xyz/wagyourtail/unimined/api/minecraft/MinecraftConfig.kt
@@ -20,7 +20,6 @@ import xyz.wagyourtail.unimined.api.minecraft.resolver.MinecraftData
 import xyz.wagyourtail.unimined.api.mod.ModsConfig
 import xyz.wagyourtail.unimined.api.runs.RunsConfig
 import xyz.wagyourtail.unimined.api.source.SourceConfig
-import xyz.wagyourtail.unimined.api.minecraft.task.AbstractRemapJarTask
 import xyz.wagyourtail.unimined.api.minecraft.task.RemapJarTask
 import xyz.wagyourtail.unimined.api.minecraft.task.RemapSourcesJarTask
 import xyz.wagyourtail.unimined.api.unimined
@@ -95,6 +94,11 @@ abstract class MinecraftConfig(val project: Project, val sourceSet: SourceSet) :
      */
     var defaultRemapJar: Boolean by FinalizeOnRead(true)
 
+    /**
+     * should unimined add the default "remapSourcesJar" task to this sourceSet?
+     */
+    var defaultRemapSourcesJar: Boolean by FinalizeOnRead(false)
+
     /**
      * if the jar task for defaultRemapJar doesn't exist, should unimined create it?
      * if so, it will be created with `from(this.sourceSet.output, sourceSets.main.output)`
diff --git a/src/minecraft/kotlin/xyz/wagyourtail/unimined/internal/minecraft/MinecraftProvider.kt b/src/minecraft/kotlin/xyz/wagyourtail/unimined/internal/minecraft/MinecraftProvider.kt
index eed2a6a8..e4eeb5e4 100644
--- a/src/minecraft/kotlin/xyz/wagyourtail/unimined/internal/minecraft/MinecraftProvider.kt
+++ b/src/minecraft/kotlin/xyz/wagyourtail/unimined/internal/minecraft/MinecraftProvider.kt
@@ -450,20 +450,6 @@ open class MinecraftProvider(project: Project, sourceSet: SourceSet) : Minecraft
         }
     }
 
-    private fun applyDefaultRemapJars() {
-        applyDefaultRemapJar<RemapJarTaskImpl>("jar", ::remap) {
-            from(sourceSet.output)
-            archiveClassifier.set(sourceSet.name)
-            from(combinedWithList.map { it.second.output })
-        }
-
-        applyDefaultRemapJar<RemapSourcesJarTaskImpl>("sourcesJar", ::remapSources) {
-            from(sourceSet.allSource)
-            archiveClassifier.set("${sourceSet.name}-sources")
-            from(combinedWithList.map { it.second.allSource })
-        }
-    }
-
     private inline fun <reified T> applyDefaultRemapJar(
         inputTaskName: String,
         remappingFunction: (Task, JarInterface<AbstractRemapJarTask>.() -> Unit) -> Unit,
@@ -611,7 +597,19 @@ open class MinecraftProvider(project: Project, sourceSet: SourceSet) : Minecraft
         if (mcPatcher.addVanillaLibraries) addLibraries(minecraftData.metadata.libraries)
 
         if (defaultRemapJar) {
-            applyDefaultRemapJars()
+            applyDefaultRemapJar<RemapJarTaskImpl>("jar", ::remap) {
+                from(sourceSet.output)
+                archiveClassifier.set(sourceSet.name)
+                from(combinedWithList.map { it.second.output })
+            }
+        }
+
+        if (defaultRemapSourcesJar) {
+            applyDefaultRemapJar<RemapSourcesJarTaskImpl>("sourcesJar", ::remapSources) {
+                from(sourceSet.allSource)
+                archiveClassifier.set("${sourceSet.name}-sources")
+                from(combinedWithList.map { it.second.allSource })
+            }
         }
 
         // apply minecraft patcher changes