From bed2031f3b41a622d86149ddcd568886e263040a Mon Sep 17 00:00:00 2001 From: modmuss50 Date: Wed, 30 Jun 2021 22:33:17 +0100 Subject: Update ASM and nightly gradle --- build.gradle | 10 +++++----- .../groovy/net/fabricmc/loom/test/util/ProjectTestTrait.groovy | 2 +- 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/build.gradle b/build.gradle index a7e766fc..1b544ada 100644 --- a/build.gradle +++ b/build.gradle @@ -65,11 +65,11 @@ dependencies { implementation ('com.google.code.gson:gson:2.8.6') implementation ('com.fasterxml.jackson.core:jackson-databind:2.12.3') implementation ('com.google.guava:guava:30.1-jre') - implementation ('org.ow2.asm:asm:9.1') - implementation ('org.ow2.asm:asm-analysis:9.1') - implementation ('org.ow2.asm:asm-commons:9.1') - implementation ('org.ow2.asm:asm-tree:9.1') - implementation ('org.ow2.asm:asm-util:9.1') + implementation ('org.ow2.asm:asm:9.2') + implementation ('org.ow2.asm:asm-analysis:9.2') + implementation ('org.ow2.asm:asm-commons:9.2') + implementation ('org.ow2.asm:asm-tree:9.2') + implementation ('org.ow2.asm:asm-util:9.2') // game handling utils implementation ('net.fabricmc:stitch:0.6.1') { diff --git a/src/test/groovy/net/fabricmc/loom/test/util/ProjectTestTrait.groovy b/src/test/groovy/net/fabricmc/loom/test/util/ProjectTestTrait.groovy index 0f72662d..5b0e105c 100644 --- a/src/test/groovy/net/fabricmc/loom/test/util/ProjectTestTrait.groovy +++ b/src/test/groovy/net/fabricmc/loom/test/util/ProjectTestTrait.groovy @@ -29,7 +29,7 @@ import org.gradle.testkit.runner.GradleRunner trait ProjectTestTrait { final static String DEFAULT_GRADLE = "7.0.1" - final static String PRE_RELEASE_GRADLE = "7.2-20210612220215+0000" + final static String PRE_RELEASE_GRADLE = "7.2-20210629235357+0000" static File gradleHome = File.createTempDir() File testProjectDir = File.createTempDir() -- cgit From 5bccf437619d4e60340e9a6a8c47ae3eb629a2d6 Mon Sep 17 00:00:00 2001 From: modmuss50 Date: Wed, 30 Jun 2021 22:54:42 +0100 Subject: Update ReproducibleBuildTest hashes, the tiny remapper update caused this. --- .../net/fabricmc/loom/test/integration/ReproducibleBuildTest.groovy | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/test/groovy/net/fabricmc/loom/test/integration/ReproducibleBuildTest.groovy b/src/test/groovy/net/fabricmc/loom/test/integration/ReproducibleBuildTest.groovy index 1860450c..6a83f2bb 100644 --- a/src/test/groovy/net/fabricmc/loom/test/integration/ReproducibleBuildTest.groovy +++ b/src/test/groovy/net/fabricmc/loom/test/integration/ReproducibleBuildTest.groovy @@ -49,8 +49,8 @@ class ReproducibleBuildTest extends Specification implements ProjectTestTrait { getOutputHash("fabric-example-mod-1.0.0-sources.jar") in sourceHash // Done for different line endings. where: gradle | modHash | sourceHash - DEFAULT_GRADLE | "6132ffb4117adb7e258f663110552952" | ["be31766e6cafbe4ae3bca9e35ba63169", "7348b0bd87d36d7ec6f3bca9c2b66062"] - PRE_RELEASE_GRADLE | "6132ffb4117adb7e258f663110552952" | ["be31766e6cafbe4ae3bca9e35ba63169", "7348b0bd87d36d7ec6f3bca9c2b66062"] + DEFAULT_GRADLE | "0f954aa060fd8fc005e834c7cd271303" | ["be31766e6cafbe4ae3bca9e35ba63169", "7348b0bd87d36d7ec6f3bca9c2b66062"] + PRE_RELEASE_GRADLE | "0f954aa060fd8fc005e834c7cd271303" | ["be31766e6cafbe4ae3bca9e35ba63169", "7348b0bd87d36d7ec6f3bca9c2b66062"] } String getOutputHash(String name) { -- cgit From 3a51777b4bbb9b27bcb5cfd8fc7b0f66660e6f62 Mon Sep 17 00:00:00 2001 From: modmuss50 Date: Sun, 4 Jul 2021 00:58:09 +0100 Subject: Use a detached configuration to add jetbrains annotations to the source remapper classpath. --- .../net/fabricmc/loom/configuration/providers/LaunchProvider.java | 3 +-- src/main/java/net/fabricmc/loom/util/SourceRemapper.java | 6 +++--- src/test/groovy/net/fabricmc/loom/test/util/ProjectTestTrait.groovy | 2 +- 3 files changed, 5 insertions(+), 6 deletions(-) diff --git a/src/main/java/net/fabricmc/loom/configuration/providers/LaunchProvider.java b/src/main/java/net/fabricmc/loom/configuration/providers/LaunchProvider.java index 980cecf0..ce5320c1 100644 --- a/src/main/java/net/fabricmc/loom/configuration/providers/LaunchProvider.java +++ b/src/main/java/net/fabricmc/loom/configuration/providers/LaunchProvider.java @@ -48,7 +48,6 @@ import net.fabricmc.loom.configuration.RemappedConfigurationEntry; import net.fabricmc.loom.util.Constants; public class LaunchProvider extends DependencyProvider { - public Dependency annotationDependency; public LaunchProvider(Project project) { super(project); @@ -81,7 +80,7 @@ public class LaunchProvider extends DependencyProvider { addDependency(Constants.Dependencies.DEV_LAUNCH_INJECTOR + Constants.Dependencies.Versions.DEV_LAUNCH_INJECTOR, Constants.Configurations.LOOM_DEVELOPMENT_DEPENDENCIES); addDependency(Constants.Dependencies.TERMINAL_CONSOLE_APPENDER + Constants.Dependencies.Versions.TERMINAL_CONSOLE_APPENDER, Constants.Configurations.LOOM_DEVELOPMENT_DEPENDENCIES); - annotationDependency = addDependency(Constants.Dependencies.JETBRAINS_ANNOTATIONS + Constants.Dependencies.Versions.JETBRAINS_ANNOTATIONS, JavaPlugin.COMPILE_ONLY_CONFIGURATION_NAME); + addDependency(Constants.Dependencies.JETBRAINS_ANNOTATIONS + Constants.Dependencies.Versions.JETBRAINS_ANNOTATIONS, JavaPlugin.COMPILE_ONLY_CONFIGURATION_NAME); postPopulationScheduler.accept(this::writeRemapClassPath); } diff --git a/src/main/java/net/fabricmc/loom/util/SourceRemapper.java b/src/main/java/net/fabricmc/loom/util/SourceRemapper.java index 1e3eb801..ae750c7f 100644 --- a/src/main/java/net/fabricmc/loom/util/SourceRemapper.java +++ b/src/main/java/net/fabricmc/loom/util/SourceRemapper.java @@ -189,9 +189,9 @@ public class SourceRemapper { m.getClassPath().add(extension.getMinecraftMappedProvider().getMappedJar().toPath()); m.getClassPath().add(extension.getMinecraftMappedProvider().getIntermediaryJar().toPath()); - Dependency annotationDependency = extension.getDependencyManager().getProvider(LaunchProvider.class).annotationDependency; - Set files = project.getConfigurations().getByName(JavaPlugin.COMPILE_CLASSPATH_CONFIGURATION_NAME) - .files(annotationDependency); + Set files = project.getConfigurations() + .detachedConfiguration(project.getDependencies().create(Constants.Dependencies.JETBRAINS_ANNOTATIONS + Constants.Dependencies.Versions.JETBRAINS_ANNOTATIONS)) + .resolve(); for (File file : files) { m.getClassPath().add(file.toPath()); diff --git a/src/test/groovy/net/fabricmc/loom/test/util/ProjectTestTrait.groovy b/src/test/groovy/net/fabricmc/loom/test/util/ProjectTestTrait.groovy index 5b0e105c..58e6cb92 100644 --- a/src/test/groovy/net/fabricmc/loom/test/util/ProjectTestTrait.groovy +++ b/src/test/groovy/net/fabricmc/loom/test/util/ProjectTestTrait.groovy @@ -29,7 +29,7 @@ import org.gradle.testkit.runner.GradleRunner trait ProjectTestTrait { final static String DEFAULT_GRADLE = "7.0.1" - final static String PRE_RELEASE_GRADLE = "7.2-20210629235357+0000" + final static String PRE_RELEASE_GRADLE = "7.2-20210703221006+0000" static File gradleHome = File.createTempDir() File testProjectDir = File.createTempDir() -- cgit From 8ce991dadcbfad4c135c0c7c8897f77e50194bb2 Mon Sep 17 00:00:00 2001 From: modmuss50 Date: Sun, 4 Jul 2021 01:00:18 +0100 Subject: Fix build... --- .../java/net/fabricmc/loom/configuration/providers/LaunchProvider.java | 2 -- src/main/java/net/fabricmc/loom/util/SourceRemapper.java | 3 --- 2 files changed, 5 deletions(-) diff --git a/src/main/java/net/fabricmc/loom/configuration/providers/LaunchProvider.java b/src/main/java/net/fabricmc/loom/configuration/providers/LaunchProvider.java index ce5320c1..b1d4c96d 100644 --- a/src/main/java/net/fabricmc/loom/configuration/providers/LaunchProvider.java +++ b/src/main/java/net/fabricmc/loom/configuration/providers/LaunchProvider.java @@ -40,7 +40,6 @@ import java.util.stream.Collectors; import org.apache.commons.io.FileUtils; import org.gradle.api.Project; -import org.gradle.api.artifacts.Dependency; import org.gradle.api.plugins.JavaPlugin; import net.fabricmc.loom.configuration.DependencyProvider; @@ -48,7 +47,6 @@ import net.fabricmc.loom.configuration.RemappedConfigurationEntry; import net.fabricmc.loom.util.Constants; public class LaunchProvider extends DependencyProvider { - public LaunchProvider(Project project) { super(project); } diff --git a/src/main/java/net/fabricmc/loom/util/SourceRemapper.java b/src/main/java/net/fabricmc/loom/util/SourceRemapper.java index ae750c7f..bd3e6aac 100644 --- a/src/main/java/net/fabricmc/loom/util/SourceRemapper.java +++ b/src/main/java/net/fabricmc/loom/util/SourceRemapper.java @@ -37,13 +37,10 @@ import org.cadixdev.lorenz.MappingSet; import org.cadixdev.mercury.Mercury; import org.cadixdev.mercury.remapper.MercuryRemapper; import org.gradle.api.Project; -import org.gradle.api.artifacts.Dependency; -import org.gradle.api.plugins.JavaPlugin; import org.zeroturnaround.zip.ZipUtil; import net.fabricmc.loom.LoomGradleExtension; import net.fabricmc.loom.configuration.RemappedConfigurationEntry; -import net.fabricmc.loom.configuration.providers.LaunchProvider; import net.fabricmc.loom.configuration.providers.mappings.MappingsProviderImpl; import net.fabricmc.loom.util.gradle.ProgressLogger; import net.fabricmc.lorenztiny.TinyMappingsReader; -- cgit From b24e067215bd36f128210830c508535135a41c58 Mon Sep 17 00:00:00 2001 From: modmuss50 Date: Mon, 5 Jul 2021 23:04:08 +0100 Subject: Fix yet another gradle deprecation, expand tests to cover it --- src/main/java/net/fabricmc/loom/task/UnpickJarTask.java | 2 +- .../net/fabricmc/loom/test/integration/UnpickTest.groovy | 12 ++++++++++-- .../net/fabricmc/loom/test/util/ProjectTestTrait.groovy | 2 +- 3 files changed, 12 insertions(+), 4 deletions(-) diff --git a/src/main/java/net/fabricmc/loom/task/UnpickJarTask.java b/src/main/java/net/fabricmc/loom/task/UnpickJarTask.java index e27d2ff2..7f66aa61 100644 --- a/src/main/java/net/fabricmc/loom/task/UnpickJarTask.java +++ b/src/main/java/net/fabricmc/loom/task/UnpickJarTask.java @@ -47,7 +47,7 @@ public class UnpickJarTask extends JavaExec { public UnpickJarTask() { getOutputs().upToDateWhen(e -> false); classpath(getProject().getConfigurations().getByName(Constants.Configurations.UNPICK_CLASSPATH)); - setMain("daomephsta.unpick.cli.Main"); + getMainClass().set("daomephsta.unpick.cli.Main"); } @Override diff --git a/src/test/groovy/net/fabricmc/loom/test/integration/UnpickTest.groovy b/src/test/groovy/net/fabricmc/loom/test/integration/UnpickTest.groovy index 2be479ea..d7717a2d 100644 --- a/src/test/groovy/net/fabricmc/loom/test/integration/UnpickTest.groovy +++ b/src/test/groovy/net/fabricmc/loom/test/integration/UnpickTest.groovy @@ -42,17 +42,25 @@ class UnpickTest extends Specification implements ProjectTestTrait { def "unpick decompile"() { when: - def result = create("genSources") + def result = create("genSources", gradle) then: result.task(":genSources").outcome == SUCCESS getClassSource("net/minecraft/block/CakeBlock.java").contains("Block.DEFAULT_SET_BLOCK_STATE_FLAG") + where: + gradle | _ + DEFAULT_GRADLE | _ + PRE_RELEASE_GRADLE | _ } def "unpick build"() { when: - def result = create("build") + def result = create("build", gradle) then: result.task(":build").outcome == SUCCESS + where: + gradle | _ + DEFAULT_GRADLE | _ + PRE_RELEASE_GRADLE | _ } String getClassSource(String classname, String mappings = MAPPINGS) { diff --git a/src/test/groovy/net/fabricmc/loom/test/util/ProjectTestTrait.groovy b/src/test/groovy/net/fabricmc/loom/test/util/ProjectTestTrait.groovy index 58e6cb92..932229f8 100644 --- a/src/test/groovy/net/fabricmc/loom/test/util/ProjectTestTrait.groovy +++ b/src/test/groovy/net/fabricmc/loom/test/util/ProjectTestTrait.groovy @@ -29,7 +29,7 @@ import org.gradle.testkit.runner.GradleRunner trait ProjectTestTrait { final static String DEFAULT_GRADLE = "7.0.1" - final static String PRE_RELEASE_GRADLE = "7.2-20210703221006+0000" + final static String PRE_RELEASE_GRADLE = "7.2-20210704221017+0000" static File gradleHome = File.createTempDir() File testProjectDir = File.createTempDir() -- cgit