simplify dependency configuration

2025-07-03 08:25:31 +00:00 · 2022-05-05 08:25:30 -07:00 · 2022-05-05 08:25:30 -07:00 · 41e7ad35bd
commit 41e7ad35bd
parent 42fb3642f9
40 changed files with 72 additions and 88 deletions
--- a/build.gradle.kts
+++ b/build.gradle.kts
@ -45,7 +45,7 @@ afterEvaluate {
    forSubProjects(":common:addons") {
        dependencies {
            "compileOnly"(project(":common:api"))
-            "shadedImplementation"("net.jafama", "jafama", Versions.Libraries.Internal.jafama)
+            "implementation"("net.jafama", "jafama", Versions.Libraries.Internal.jafama)
            "testImplementation"("net.jafama", "jafama", Versions.Libraries.Internal.jafama)
            "testImplementation"(project(":common:api"))
        }
--- a/buildSrc/src/main/kotlin/DependencyConfig.kt
+++ b/buildSrc/src/main/kotlin/DependencyConfig.kt
@ -6,25 +6,9 @@ import org.gradle.kotlin.dsl.getting
 import org.gradle.kotlin.dsl.repositories

 fun Project.configureDependencies() {
-    val api by configurations.getting
-    val implementation by configurations.getting
    val testImplementation by configurations.getting
    val compileOnly by configurations.getting
    
-    val shaded by configurations.creating
-    
-    @Suppress("UNUSED_VARIABLE")
-    val shadedApi by configurations.creating {
-        shaded.extendsFrom(this)
-        api.extendsFrom(this)
-    }
-    
-    @Suppress("UNUSED_VARIABLE")
-    val shadedImplementation by configurations.creating {
-        shaded.extendsFrom(this)
-        implementation.extendsFrom(this)
-    }
-    
    repositories {
        mavenCentral()
        gradlePluginPortal()
--- a/buildSrc/src/main/kotlin/DistributionConfig.kt
+++ b/buildSrc/src/main/kotlin/DistributionConfig.kt
@ -122,8 +122,6 @@ fun Project.configureDistribution() {
        // Tell shadow to download the packs
        dependsOn(downloadDefaultPacks)
        
-        configurations = listOf(project.configurations["shaded"])
-        
        archiveClassifier.set("shaded")
        setVersion(project.version)
        relocate("org.apache.commons", "com.dfsek.terra.lib.commons")
--- a/common/addons/biome-provider-image/build.gradle.kts
+++ b/common/addons/biome-provider-image/build.gradle.kts
@ -1,5 +1,5 @@
 version = version("0.1.0")

 dependencies {
-    shadedApi(project(":common:addons:manifest-addon-loader"))
+    compileOnlyApi(project(":common:addons:manifest-addon-loader"))
 }
--- a/common/addons/biome-provider-pipeline/build.gradle.kts
+++ b/common/addons/biome-provider-pipeline/build.gradle.kts
@ -1,5 +1,5 @@
 version = version("0.1.0")

 dependencies {
-    shadedApi(project(":common:addons:manifest-addon-loader"))
+    compileOnlyApi(project(":common:addons:manifest-addon-loader"))
 }
--- a/common/addons/biome-provider-single/build.gradle.kts
+++ b/common/addons/biome-provider-single/build.gradle.kts
@ -1,5 +1,5 @@
 version = version("0.1.0")

 dependencies {
-    shadedApi(project(":common:addons:manifest-addon-loader"))
+    compileOnlyApi(project(":common:addons:manifest-addon-loader"))
 }
--- a/common/addons/chunk-generator-noise-3d/build.gradle.kts
+++ b/common/addons/chunk-generator-noise-3d/build.gradle.kts
@ -1,5 +1,5 @@
 version = version("0.1.0")

 dependencies {
-    shadedApi(project(":common:addons:manifest-addon-loader"))
+    compileOnlyApi(project(":common:addons:manifest-addon-loader"))
 }
--- a/common/addons/command-addons/build.gradle.kts
+++ b/common/addons/command-addons/build.gradle.kts
@ -1,5 +1,5 @@
 version = version("0.1.0")

 dependencies {
-    shadedApi(project(":common:addons:manifest-addon-loader"))
+    compileOnlyApi(project(":common:addons:manifest-addon-loader"))
 }
--- a/common/addons/command-packs/build.gradle.kts
+++ b/common/addons/command-packs/build.gradle.kts
@ -1,5 +1,5 @@
 version = version("0.1.0")

 dependencies {
-    shadedApi(project(":common:addons:manifest-addon-loader"))
+    compileOnlyApi(project(":common:addons:manifest-addon-loader"))
 }
--- a/common/addons/command-profiler/build.gradle.kts
+++ b/common/addons/command-profiler/build.gradle.kts
@ -1,5 +1,5 @@
 version = version("0.1.0")

 dependencies {
-    shadedApi(project(":common:addons:manifest-addon-loader"))
+    compileOnlyApi(project(":common:addons:manifest-addon-loader"))
 }
--- a/common/addons/command-structures/build.gradle.kts
+++ b/common/addons/command-structures/build.gradle.kts
@ -1,5 +1,5 @@
 version = version("0.1.0")

 dependencies {
-    shadedApi(project(":common:addons:manifest-addon-loader"))
+    compileOnlyApi(project(":common:addons:manifest-addon-loader"))
 }
--- a/common/addons/config-biome/build.gradle.kts
+++ b/common/addons/config-biome/build.gradle.kts
@ -1,5 +1,5 @@
 version = version("0.1.0")

 dependencies {
-    shadedApi(project(":common:addons:manifest-addon-loader"))
+    compileOnlyApi(project(":common:addons:manifest-addon-loader"))
 }
--- a/common/addons/config-distributors/build.gradle.kts
+++ b/common/addons/config-distributors/build.gradle.kts
@ -1,5 +1,5 @@
 version = version("0.1.0")

 dependencies {
-    shadedApi(project(":common:addons:manifest-addon-loader"))
+    compileOnlyApi(project(":common:addons:manifest-addon-loader"))
 }
--- a/common/addons/config-feature/build.gradle.kts
+++ b/common/addons/config-feature/build.gradle.kts
@ -1,5 +1,5 @@
 version = version("0.1.0")

 dependencies {
-    shadedApi(project(":common:addons:manifest-addon-loader"))
+    compileOnlyApi(project(":common:addons:manifest-addon-loader"))
 }
--- a/common/addons/config-flora/build.gradle.kts
+++ b/common/addons/config-flora/build.gradle.kts
@ -1,5 +1,5 @@
 version = version("0.1.0")

 dependencies {
-    shadedApi(project(":common:addons:manifest-addon-loader"))
+    compileOnlyApi(project(":common:addons:manifest-addon-loader"))
 }
--- a/common/addons/config-locators/build.gradle.kts
+++ b/common/addons/config-locators/build.gradle.kts
@ -1,5 +1,5 @@
 version = version("0.1.0")

 dependencies {
-    shadedApi(project(":common:addons:manifest-addon-loader"))
+    compileOnlyApi(project(":common:addons:manifest-addon-loader"))
 }
--- a/common/addons/config-noise-function/build.gradle.kts
+++ b/common/addons/config-noise-function/build.gradle.kts
@ -1,5 +1,5 @@
 version = version("0.1.0")

 dependencies {
-    shadedApi(project(":common:addons:manifest-addon-loader"))
+    compileOnlyApi(project(":common:addons:manifest-addon-loader"))
 }
--- a/common/addons/config-ore/build.gradle.kts
+++ b/common/addons/config-ore/build.gradle.kts
@ -1,5 +1,5 @@
 version = version("0.1.0")

 dependencies {
-    shadedApi(project(":common:addons:manifest-addon-loader"))
+    compileOnlyApi(project(":common:addons:manifest-addon-loader"))
 }
--- a/common/addons/config-palette/build.gradle.kts
+++ b/common/addons/config-palette/build.gradle.kts
@ -1,5 +1,5 @@
 version = version("0.1.0")

 dependencies {
-    shadedApi(project(":common:addons:manifest-addon-loader"))
+    compileOnlyApi(project(":common:addons:manifest-addon-loader"))
 }
--- a/common/addons/config-structure/build.gradle.kts
+++ b/common/addons/config-structure/build.gradle.kts
@ -1,6 +1,6 @@
 version = version("0.1.0")

 dependencies {
-    shadedApi("com.googlecode.json-simple:json-simple:1.1.1")
-    shadedApi(project(":common:addons:manifest-addon-loader"))
+    api("com.googlecode.json-simple:json-simple:1.1.1")
+    compileOnlyApi(project(":common:addons:manifest-addon-loader"))
 }
--- a/common/addons/generation-stage-feature/build.gradle.kts
+++ b/common/addons/generation-stage-feature/build.gradle.kts
@ -1,5 +1,5 @@
 version = version("0.1.0")

 dependencies {
-    shadedApi(project(":common:addons:manifest-addon-loader"))
+    compileOnlyApi(project(":common:addons:manifest-addon-loader"))
 }
--- a/common/addons/generation-stage-structure/build.gradle.kts
+++ b/common/addons/generation-stage-structure/build.gradle.kts
@ -1,5 +1,5 @@
 version = version("0.1.0")

 dependencies {
-    shadedApi(project(":common:addons:manifest-addon-loader"))
+    compileOnlyApi(project(":common:addons:manifest-addon-loader"))
 }
--- a/common/addons/language-yaml/build.gradle.kts
+++ b/common/addons/language-yaml/build.gradle.kts
@ -1,6 +1,6 @@
 version = version("0.1.0")

 dependencies {
-    shadedImplementation("com.dfsek.tectonic:yaml:${Versions.Libraries.tectonic}")
-    shadedApi(project(":common:addons:manifest-addon-loader"))
+    implementation("com.dfsek.tectonic:yaml:${Versions.Libraries.tectonic}")
+    compileOnlyApi(project(":common:addons:manifest-addon-loader"))
 }
--- a/common/addons/manifest-addon-loader/build.gradle.kts
+++ b/common/addons/manifest-addon-loader/build.gradle.kts
@ -1,8 +1,8 @@
 version = version("0.1.0")

 dependencies {
-    shadedApi("commons-io:commons-io:2.6")
-    shadedImplementation("com.dfsek.tectonic:yaml:${Versions.Libraries.tectonic}")
+    api("commons-io:commons-io:2.7")
+    implementation("com.dfsek.tectonic:yaml:${Versions.Libraries.tectonic}")
 }

 tasks.withType<Jar> {
--- a/common/addons/palette-block-shortcut/build.gradle.kts
+++ b/common/addons/palette-block-shortcut/build.gradle.kts
@ -1,5 +1,5 @@
 version = version("0.1.0")

 dependencies {
-    shadedApi(project(":common:addons:manifest-addon-loader"))
+    compileOnlyApi(project(":common:addons:manifest-addon-loader"))
 }
--- a/common/addons/structure-block-shortcut/build.gradle.kts
+++ b/common/addons/structure-block-shortcut/build.gradle.kts
@ -1,5 +1,5 @@
 version = version("0.1.0")

 dependencies {
-    shadedApi(project(":common:addons:manifest-addon-loader"))
+    compileOnlyApi(project(":common:addons:manifest-addon-loader"))
 }
--- a/common/addons/structure-mutator/build.gradle.kts
+++ b/common/addons/structure-mutator/build.gradle.kts
@ -1,5 +1,5 @@
 version = version("0.1.0")

 dependencies {
-    shadedApi(project(":common:addons:manifest-addon-loader"))
+    compileOnlyApi(project(":common:addons:manifest-addon-loader"))
 }
--- a/common/addons/structure-sponge-loader/build.gradle.kts
+++ b/common/addons/structure-sponge-loader/build.gradle.kts
@ -11,9 +11,9 @@ repositories {
 }

 dependencies {
-    shadedApi("commons-io:commons-io:2.6")
-    shadedApi("com.github.Querz:NBT:6.1")
-    shadedApi(project(":common:addons:manifest-addon-loader"))
+    api("commons-io:commons-io:2.6")
+    api("com.github.Querz:NBT:6.1")
+    compileOnlyApi(project(":common:addons:manifest-addon-loader"))
 }

 tasks.named<ShadowJar>("shadowJar") {
--- a/common/addons/structure-terrascript-loader/build.gradle.kts
+++ b/common/addons/structure-terrascript-loader/build.gradle.kts
@ -8,8 +8,8 @@ plugins {
 }

 dependencies {
-    shadedApi("commons-io:commons-io:2.6")
-    shadedApi(project(":common:addons:manifest-addon-loader"))
+    api("commons-io:commons-io:2.6")
+    compileOnlyApi(project(":common:addons:manifest-addon-loader"))
 }

 tasks.named<ShadowJar>("shadowJar") {
--- a/common/addons/terrascript-function-check-noise-3d/build.gradle.kts
+++ b/common/addons/terrascript-function-check-noise-3d/build.gradle.kts
@ -1,7 +1,7 @@
 version = version("0.1.0")

 dependencies {
-    shadedApi(project(":common:addons:manifest-addon-loader"))
-    shadedApi(project(":common:addons:chunk-generator-noise-3d"))
-    shadedApi(project(":common:addons:structure-terrascript-loader"))
+    compileOnlyApi(project(":common:addons:manifest-addon-loader"))
+    compileOnlyApi(project(":common:addons:chunk-generator-noise-3d"))
+    compileOnlyApi(project(":common:addons:structure-terrascript-loader"))
 }
--- a/common/addons/terrascript-function-sampler/build.gradle.kts
+++ b/common/addons/terrascript-function-sampler/build.gradle.kts
@ -1,7 +1,7 @@
 version = version("0.1.0")

 dependencies {
-    shadedApi(project(":common:addons:manifest-addon-loader"))
-    shadedApi(project(":common:addons:config-noise-function"))
-    shadedApi(project(":common:addons:structure-terrascript-loader"))
+    compileOnlyApi(project(":common:addons:manifest-addon-loader"))
+    compileOnlyApi(project(":common:addons:config-noise-function"))
+    compileOnlyApi(project(":common:addons:structure-terrascript-loader"))
 }
--- a/common/api/build.gradle.kts
+++ b/common/api/build.gradle.kts
@ -1,11 +1,11 @@
 dependencies {
-    shadedApi("ca.solo-studios", "strata", Versions.Libraries.strata)
-    shadedApi("org.slf4j", "slf4j-api", Versions.Libraries.slf4j)
-    shadedApi("cloud.commandframework", "cloud-core", Versions.Libraries.cloud)
+    api("ca.solo-studios", "strata", Versions.Libraries.strata)
+    api("org.slf4j", "slf4j-api", Versions.Libraries.slf4j)
+    api("cloud.commandframework", "cloud-core", Versions.Libraries.cloud)
    
-    shadedApi("com.dfsek", "paralithic", Versions.Libraries.paralithic)
-    shadedApi("com.dfsek.tectonic", "common", Versions.Libraries.tectonic)
+    api("com.dfsek", "paralithic", Versions.Libraries.paralithic)
+    api("com.dfsek.tectonic", "common", Versions.Libraries.tectonic)
    
    
-    shadedImplementation("net.jafama", "jafama", Versions.Libraries.Internal.jafama)
+    implementation("net.jafama", "jafama", Versions.Libraries.Internal.jafama)
 }
--- a/common/implementation/base/build.gradle.kts
+++ b/common/implementation/base/build.gradle.kts
@ -1,13 +1,13 @@
 dependencies {
-    shadedApi(project(":common:api"))
-    shadedApi(project(":common:implementation:bootstrap-addon-loader"))
+    api(project(":common:api"))
+    api(project(":common:implementation:bootstrap-addon-loader"))
    
-    shadedImplementation("commons-io", "commons-io", Versions.Libraries.Internal.apacheIO)
+    implementation("commons-io", "commons-io", Versions.Libraries.Internal.apacheIO)
    
-    shadedImplementation("org.apache.commons", "commons-text", Versions.Libraries.Internal.apacheText)
-    shadedImplementation("com.dfsek.tectonic", "yaml", Versions.Libraries.tectonic)
+    implementation("org.apache.commons", "commons-text", Versions.Libraries.Internal.apacheText)
+    implementation("com.dfsek.tectonic", "yaml", Versions.Libraries.tectonic)
    
-    shadedImplementation("net.jafama", "jafama", Versions.Libraries.Internal.jafama)
+    implementation("net.jafama", "jafama", Versions.Libraries.Internal.jafama)
    
    implementation("org.ow2.asm", "asm", Versions.Libraries.Internal.asm)
 }
--- a/common/implementation/bootstrap-addon-loader/build.gradle.kts
+++ b/common/implementation/bootstrap-addon-loader/build.gradle.kts
@ -1,3 +1,3 @@
 dependencies {
-    "shadedApi"(project(":common:api"))
+    compileOnlyApi(project(":common:api"))
 }
--- a/platforms/bukkit/build.gradle.kts
+++ b/platforms/bukkit/build.gradle.kts
@ -14,21 +14,21 @@ val paperURL = "https://papermc.io/api/v2/projects/paper/versions/%version%/buil
 val purpurURL = "https://api.pl3x.net/v2/purpur/%version%/latest/download"

 dependencies {
-    shadedApi(project(":common:implementation:base"))
+    api(project(":common:implementation:base"))
    
-    shadedApi("org.slf4j:slf4j-api:1.8.0-beta4") {
+    api("org.slf4j:slf4j-api:1.8.0-beta4") {
        because("Minecraft 1.17+ includes slf4j 1.8.0-beta4, so we need to shade it for other versions.")
    }
-    shadedImplementation("org.apache.logging.log4j", "log4j-slf4j18-impl", Versions.Libraries.log4j_slf4j_impl) {
+    implementation("org.apache.logging.log4j", "log4j-slf4j18-impl", Versions.Libraries.log4j_slf4j_impl) {
        because("Minecraft 1.17+ includes slf4j 1.8.0-beta4, so we need to shade it for other versions.")
    }
    
    compileOnly("io.papermc.paper", "paper-api", Versions.Bukkit.paper)
-    shadedImplementation("io.papermc", "paperlib", Versions.Bukkit.paperLib)
+    implementation("io.papermc", "paperlib", Versions.Bukkit.paperLib)
    
-    shadedApi("com.google.guava:guava:30.0-jre")
+    api("com.google.guava:guava:30.0-jre")
    
-    shadedApi("cloud.commandframework", "cloud-paper", Versions.Libraries.cloud)
+    api("cloud.commandframework", "cloud-paper", Versions.Libraries.cloud)
 }

 val jvmFlags = listOf(
--- a/platforms/cli/build.gradle.kts
+++ b/platforms/cli/build.gradle.kts
@ -3,15 +3,15 @@ repositories {
 }

 dependencies {
-    shadedApi("commons-io:commons-io:2.6")
-    shadedApi("com.github.Querz:NBT:6.1")
-    shadedApi(project(":common:implementation:base"))
+    api("commons-io:commons-io:2.7")
+    api("com.github.Querz:NBT:6.1")
+    api(project(":common:implementation:base"))
    
-    shadedImplementation("com.google.guava:guava:31.0.1-jre")
+    implementation("com.google.guava:guava:31.0.1-jre")
    
-    shadedImplementation("ch.qos.logback:logback-classic:1.2.7")
+    implementation("ch.qos.logback:logback-classic:1.2.9")
    
-    shadedImplementation("net.jafama", "jafama", Versions.Libraries.Internal.jafama)
+    implementation("net.jafama", "jafama", Versions.Libraries.Internal.jafama)
 }

 tasks.withType<Jar>() {
--- a/platforms/fabric/build.gradle.kts
+++ b/platforms/fabric/build.gradle.kts
@ -8,7 +8,7 @@ plugins {
 }

 dependencies {
-    shadedApi(project(":common:implementation:base"))
+    api(project(":common:implementation:base"))
    
    minecraft("com.mojang:minecraft:${Versions.Fabric.minecraft}")
    mappings("net.fabricmc:yarn:${Versions.Fabric.yarn}:v2")
--- a/platforms/fabric/src/main/java/com/dfsek/terra/fabric/mixin/lifecycle/DataPackContentsMixin.java
+++ b/platforms/fabric/src/main/java/com/dfsek/terra/fabric/mixin/lifecycle/DataPackContentsMixin.java
@ -13,7 +13,9 @@ import com.dfsek.terra.fabric.util.FabricUtil;

@Mixin(DataPackContents.class)
 public class DataPackContentsMixin {
-    
+    /*
+     * #refresh populates all tags in the registries
+     */
    @Inject(method = "refresh(Lnet/minecraft/util/registry/DynamicRegistryManager;)V", at = @At("RETURN"))
    private void injectReload(DynamicRegistryManager dynamicRegistryManager, CallbackInfo ci) {
        FabricUtil.registerTags(dynamicRegistryManager.get(Registry.BIOME_KEY));
--- a/platforms/merged/build.gradle.kts
+++ b/platforms/merged/build.gradle.kts
@ -3,7 +3,7 @@ val platformOverrides = mapOf(
                             )

 dependencies {
-    shadedApi(project(":common:implementation"))
+    api(project(":common:implementation:base"))
 }

 val taskSet = HashSet<AbstractArchiveTask>()
--- a/platforms/sponge/build.gradle.kts
+++ b/platforms/sponge/build.gradle.kts
@ -9,12 +9,12 @@ repositories {
 }

 dependencies {
-    shadedApi(project(":common:implementation:base"))
+    api(project(":common:implementation:base"))
    
-    shadedApi("org.slf4j:slf4j-api:1.8.0-beta4") {
+    api("org.slf4j:slf4j-api:1.8.0-beta4") {
        because("Minecraft 1.17+ includes slf4j 1.8.0-beta4, so we need to shade it for other versions.")
    }
-    shadedImplementation("org.apache.logging.log4j", "log4j-slf4j18-impl", Versions.Libraries.log4j_slf4j_impl) {
+    implementation("org.apache.logging.log4j", "log4j-slf4j18-impl", Versions.Libraries.log4j_slf4j_impl) {
        because("Minecraft 1.17+ includes slf4j 1.8.0-beta4, so we need to shade it for other versions.")
    }