imports/architectury-loom
1
0
Fork 0
mirror of https://github.com/architectury/architectury-loom.git synced 2026-04-02 13:37:45 -05:00
Code Issues Packages Projects Releases Wiki Activity

Add warning in RemapJarTask for using mixins in Forge projects

Browse Source
This commit is contained in:
Juuxel
2020-07-30 14:44:03 +03:00
parent 186b09518b
commit a45e9f4729
1 changed files with 6 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
src/main/java/net/fabricmc/loom/task/RemapJarTask.java
View File

@@ -93,7 +93,6 @@ public class RemapJarTask extends Jar {
String fromM = "named";
String toM = "official";
// ^ This is passed to SrgRemapJarTask.
// FIXME: The task crashes because it's passed the int -> named mappings instead of full mappings
Set<File> classpathFiles = new LinkedHashSet<>(
project.getConfigurations().getByName("compileClasspath").getFiles()
@@ -107,8 +106,13 @@ public class RemapJarTask extends Jar {
remapperBuilder = remapperBuilder.withMappings(TinyRemapperMappingsHelper.create(mappingsProvider.getMappings(), fromM, toM, false));
// FIXME: The mixin map is named->intermediary, but I think we need named->srg?
if (mixinMapFile.exists()) {
remapperBuilder = remapperBuilder.withMappings(TinyUtils.createTinyMappingProvider(mixinMapPath, fromM, toM));
project.getLogger().error("Mixins in Forge projects are currently not supported.");
if (false) {
remapperBuilder = remapperBuilder.withMappings(TinyUtils.createTinyMappingProvider(mixinMapPath, fromM, toM));
}
}
project.getLogger().lifecycle(":remapping " + input.getFileName());