Skip to content

Commit e7d0a49

Browse files
committed
fix: yarn on 26.1
1 parent 83e2902 commit e7d0a49

5 files changed

Lines changed: 23 additions & 10 deletions

File tree

src/main/java/org/relativitymc/neoloom/neoforge/NFRTMergedMinecraftProvider.java

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -186,6 +186,14 @@ public List<Path> getMinecraftJars() {
186186
return List.of(this.minecraftMergedJar);
187187
}
188188

189+
public List<Path> getFullClasspath() {
190+
if (this.mergedNeoForgeJar) {
191+
return List.of(this.minecraftMergedJar);
192+
} else {
193+
return List.of(this.minecraftMergedJar, getNeoForgeUniversalJar());
194+
}
195+
}
196+
189197
@Override
190198
public File getMinecraftClientJar() {
191199
throw new UnsupportedOperationException("handled by NFRT");

src/main/java/org/relativitymc/neoloom/neoforge/mappings/FieldMappingsMigrator.java

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,6 @@
4949
import org.slf4j.LoggerFactory;
5050

5151
import net.fabricmc.loom.api.mappings.layered.MappingsNamespace;
52-
import net.fabricmc.loom.configuration.providers.minecraft.MinecraftProvider;
5352
import net.fabricmc.loom.util.Constants;
5453
import net.fabricmc.loom.util.FileSystemUtil;
5554
import net.fabricmc.mappingio.MappingReader;
@@ -61,14 +60,16 @@
6160
import dev.architectury.loom.util.Stopwatch;
6261
import dev.architectury.loom.util.ThreadingUtils;
6362

63+
import org.relativitymc.neoloom.neoforge.NFRTMergedMinecraftProvider;
64+
6465
public final class FieldMappingsMigrator implements MappingsMigrator {
6566
private static final Logger LOGGER = LoggerFactory.getLogger(FieldMappingsMigrator.class);
6667

6768
private List<Map.Entry<FieldMember, String>> migratedFields = new ArrayList<>();
6869
public Path migratedFieldsCache;
6970

7071
@Override
71-
public long setup(Project project, MinecraftProvider minecraftProvider, Path cache, Path rawMappings) throws IOException {
72+
public long setup(Project project, NFRTMergedMinecraftProvider minecraftProvider, Path cache, Path rawMappings) throws IOException {
7273
migratedFieldsCache = cache.resolve("migrated-fields.json");
7374
migratedFields.clear();
7475

src/main/java/org/relativitymc/neoloom/neoforge/mappings/ForgeMigratedMappingConfiguration.java

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -39,6 +39,8 @@
3939

4040
import dev.architectury.loom.util.Stopwatch;
4141

42+
import org.relativitymc.neoloom.neoforge.NFRTMergedMinecraftProvider;
43+
4244
public final class ForgeMigratedMappingConfiguration extends MappingConfiguration {
4345
private final List<MappingsMigrator> migrators = List.of(new FieldMappingsMigrator(), new MethodInheritanceMappingsMigrator());
4446
private final Path hashPath;
@@ -60,7 +62,7 @@ protected void manipulateMappings(@NonNull Project project) throws IOException {
6062
this.tinyMappings = mappingsWorkingDir().resolve("mappings-migrated.tiny");
6163

6264
for (MappingsMigrator migrator : this.migrators) {
63-
hash = hash * 31 + migrator.setup(project, extension.getMinecraftProvider(), this.mappingsWorkingDir(), this.rawTinyMappings);
65+
hash = hash * 31 + migrator.setup(project, (NFRTMergedMinecraftProvider) extension.getMinecraftProvider(), this.mappingsWorkingDir(), this.rawTinyMappings);
6466
}
6567

6668
if (!isOutdated(extension)) {

src/main/java/org/relativitymc/neoloom/neoforge/mappings/MappingsMigrator.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -30,10 +30,10 @@
3030

3131
import org.gradle.api.Project;
3232

33-
import net.fabricmc.loom.configuration.providers.minecraft.MinecraftProvider;
33+
import org.relativitymc.neoloom.neoforge.NFRTMergedMinecraftProvider;
3434

3535
public interface MappingsMigrator {
36-
long setup(Project project, MinecraftProvider minecraftProvider, Path cache, Path rawMappings) throws IOException;
36+
long setup(Project project, NFRTMergedMinecraftProvider minecraftProvider, Path cache, Path rawMappings) throws IOException;
3737

3838
void migrate(Project project, List<MappingsEntry> entries) throws IOException;
3939

src/main/java/org/relativitymc/neoloom/neoforge/mappings/MethodInheritanceMappingsMigrator.java

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,6 @@
4949

5050
import net.fabricmc.loom.LoomGradleExtension;
5151
import net.fabricmc.loom.api.mappings.layered.MappingsNamespace;
52-
import net.fabricmc.loom.configuration.providers.minecraft.MinecraftProvider;
5352
import net.fabricmc.loom.util.Constants;
5453
import net.fabricmc.loom.util.FileSystemUtil;
5554
import net.fabricmc.loom.util.Pair;
@@ -61,6 +60,8 @@
6160

6261
import dev.architectury.loom.util.collection.Multimap;
6362

63+
import org.relativitymc.neoloom.neoforge.NFRTMergedMinecraftProvider;
64+
6465
/**
6566
* With some forge patches, methods can inherit methods from a class that is not in the mappings.
6667
* This migrator will try to detect all the methods that are inherited from a class that is not in the mappings,
@@ -70,7 +71,7 @@ public final class MethodInheritanceMappingsMigrator implements MappingsMigrator
7071
private Set<Pair<String, String>> methodsToRemove;
7172

7273
@Override
73-
public long setup(Project project, MinecraftProvider minecraftProvider, Path cache, Path rawMappings) throws IOException {
74+
public long setup(Project project, NFRTMergedMinecraftProvider minecraftProvider, Path cache, Path rawMappings) throws IOException {
7475
Path cacheFile = cache.resolve("method-inheritance-migrator.json");
7576

7677
if (!minecraftProvider.refreshDeps() && Files.exists(cacheFile)) {
@@ -84,9 +85,10 @@ public long setup(Project project, MinecraftProvider minecraftProvider, Path cac
8485
LoomGradleExtension extension = LoomGradleExtension.get(project);
8586
methodsToRemove = new HashSet<>();
8687

87-
for (Path jar : minecraftProvider.getMinecraftJars()) {
88-
methodsToRemove.addAll(prepareCache(project.getLogger(), rawMappings, List.of(jar)));
89-
}
88+
// for (Path jar : minecraftProvider.getMinecraftJars()) {
89+
// methodsToRemove.addAll(prepareCache(project.getLogger(), rawMappings, List.of(jar)));
90+
// }
91+
methodsToRemove.addAll(prepareCache(project.getLogger(), rawMappings, minecraftProvider.getFullClasspath()));
9092

9193
Files.writeString(cacheFile, new Gson().toJson(methodsToRemove.stream().sorted(Comparator.comparing(p -> p.left() + "|" + p.right())).toList()), StandardOpenOption.CREATE, StandardOpenOption.TRUNCATE_EXISTING);
9294
}

0 commit comments

Comments
 (0)