Skip to content

Commit f884c0d

Browse files
committed
refactor: replace setup with manual userdev parsing
1 parent a331a55 commit f884c0d

File tree

11 files changed

+275
-137
lines changed

11 files changed

+275
-137
lines changed

gradle/runtime.libs.versions.toml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -37,3 +37,4 @@ enigma-swing = { module = "cuchaz:enigma-swing", version.ref = "enigma" }
3737

3838
# Forge runtime libs
3939
unprotect-fancymodloader10 = { module = "io.github.juuxel:unprotect-fancymodloader10", version.ref = "unprotect" }
40+
unprotect-modlauncher = { module = "io.github.juuxel:unprotect-modlauncher", version.ref = "unprotect" }

src/main/java/net/fabricmc/loom/configuration/LoomConfigurations.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -94,7 +94,7 @@ public void run() {
9494
registerNonTransitive(Constants.Configurations.LOADER_DEPENDENCIES, Role.RESOLVABLE);
9595

9696
registerNonTransitive(Constants.Configurations.MINECRAFT, Role.NONE);
97-
registerNonTransitive(Constants.Configurations.NEOFORGE, Role.NONE);
97+
registerNonTransitive(Constants.Configurations.FORGE_USERDEV, Role.NONE);
9898

9999
register(Constants.Configurations.NFRT_TOOL, Role.RESOLVABLE).configure(configuration -> {
100100
configuration.defaultDependencies(dependencies -> {

src/main/java/net/fabricmc/loom/configuration/providers/minecraft/MinecraftMetadataProvider.java

Lines changed: 12 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -61,12 +61,12 @@ private MinecraftMetadataProvider(Options options, Function<String, DownloadBuil
6161

6262
public static MinecraftMetadataProvider create(ConfigContext configContext) {
6363
final String minecraftVersion = resolveMinecraftVersion(configContext.project());
64-
final @Nullable ExternalModuleDependency neoForgeDependency = resolveNeoForgeDependency(configContext.project());
64+
final @Nullable ExternalModuleDependency forgeUserdevDependency = resolveForgeUserdevDependency(configContext.project());
6565

6666
return new MinecraftMetadataProvider(
6767
MinecraftMetadataProvider.Options.create(
6868
minecraftVersion,
69-
neoForgeDependency,
69+
forgeUserdevDependency,
7070
configContext.project()
7171
),
7272
configContext.extension()::download
@@ -78,8 +78,8 @@ private static String resolveMinecraftVersion(Project project) {
7878
return dependency.getDependency().getVersion();
7979
}
8080

81-
private static @Nullable ExternalModuleDependency resolveNeoForgeDependency(Project project) {
82-
final Optional<DependencyInfo> dependency = DependencyInfo.createOptional(project, Constants.Configurations.NEOFORGE);
81+
private static @Nullable ExternalModuleDependency resolveForgeUserdevDependency(Project project) {
82+
final Optional<DependencyInfo> dependency = DependencyInfo.createOptional(project, Constants.Configurations.FORGE_USERDEV);
8383
Dependency dep1 = dependency.map(dependencyInfo -> dependencyInfo.getDependency()).orElse(null);
8484

8585
if (dep1 != null && !(dep1 instanceof ExternalModuleDependency)) {
@@ -93,8 +93,8 @@ public String getMinecraftVersion() {
9393
return options.minecraftVersion();
9494
}
9595

96-
public @Nullable ExternalModuleDependency getNeoForgeDependency() {
97-
return options.neoforgeDependency();
96+
public @Nullable ExternalModuleDependency getForgeUserdevDependency() {
97+
return options.forgeUserdevDependency();
9898
}
9999

100100
public MinecraftVersionMeta getVersionMeta() {
@@ -199,28 +199,28 @@ private String getVersionMetaFileName() {
199199
}
200200

201201
public record Options(String minecraftVersion,
202-
@Nullable ExternalModuleDependency neoforgeDependency,
202+
@Nullable ExternalModuleDependency forgeUserdevDependency,
203203
ManifestLocations versionsManifests,
204204
@Nullable String customManifestUrl,
205205
Path userCache,
206-
@Nullable Path neoForgeCache,
206+
@Nullable Path forgeCache,
207207
Path workingDir) {
208-
public static Options create(String minecraftVersion, @Nullable ExternalModuleDependency neoForgeDependency, Project project) {
208+
public static Options create(String minecraftVersion, @Nullable ExternalModuleDependency forgeUserdevDependency, Project project) {
209209
final LoomGradleExtension extension = LoomGradleExtension.get(project);
210210
final Path userCache = extension.getFiles().getUserCache().toPath();
211211
final Path workingDir = MinecraftProvider.minecraftWorkingDirectory(project, minecraftVersion).toPath();
212-
final Path neoForgeCache = neoForgeDependency != null ? MinecraftProvider.neoForgeWorkingDirectory(project, minecraftVersion, neoForgeDependency).toPath() : null;
212+
final Path forgeCache = forgeUserdevDependency != null ? MinecraftProvider.forgeWorkingDirectory(project, minecraftVersion, forgeUserdevDependency).toPath() : null;
213213

214214
final ManifestLocations manifestLocations = extension.getVersionsManifests();
215215
final Property<String> customMetaUrl = extension.getCustomMinecraftMetadata();
216216

217217
return new Options(
218218
minecraftVersion,
219-
neoForgeDependency,
219+
forgeUserdevDependency,
220220
manifestLocations,
221221
customMetaUrl.getOrNull(),
222222
userCache,
223-
neoForgeCache,
223+
forgeCache,
224224
workingDir
225225
);
226226
}

src/main/java/net/fabricmc/loom/configuration/providers/minecraft/MinecraftProvider.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -310,13 +310,13 @@ public static File minecraftWorkingDirectory(Project project, String version) {
310310
return workingDir;
311311
}
312312

313-
public static File neoForgeWorkingDirectory(Project project, String minecraftVersion, ExternalModuleDependency neoForgeVersion) {
314-
File workingDir = new File(minecraftWorkingDirectory(project, minecraftVersion), mangleNeoForgeVersion(neoForgeVersion));
313+
public static File forgeWorkingDirectory(Project project, String minecraftVersion, ExternalModuleDependency neoForgeVersion) {
314+
File workingDir = new File(minecraftWorkingDirectory(project, minecraftVersion), mangleForgeVersion(neoForgeVersion));
315315
workingDir.mkdirs();
316316
return workingDir;
317317
}
318318

319-
public static String mangleNeoForgeVersion(ExternalModuleDependency neoForgeVersion) {
319+
public static String mangleForgeVersion(ExternalModuleDependency neoForgeVersion) {
320320
return neoForgeVersion.getGroup() + "." + neoForgeVersion.getName() + "_" + neoForgeVersion.getVersion();
321321
}
322322
}

src/main/java/net/fabricmc/loom/extension/LoomGradleExtensionApiImpl.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -176,7 +176,7 @@ protected LoomGradleExtensionApiImpl(Project project, LoomFiles directories) {
176176

177177
// if no configuration is selected by the user, attempt to select one
178178
// based on the mc version and which sides are present for it
179-
if (metadataProvider.getNeoForgeDependency() != null) {
179+
if (metadataProvider.getForgeUserdevDependency() != null) {
180180
return MinecraftJarConfiguration.NEOFORGE_MERGED;
181181
} else if (!metadataProvider.getVersionMeta().hasServer()) {
182182
return MinecraftJarConfiguration.CLIENT_ONLY;

src/main/java/net/fabricmc/loom/util/Constants.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,7 @@ public static final class Configurations {
5757
public static final String INCLUDE_INTERNAL = "includeInternal";
5858
public static final String MINECRAFT = "minecraft";
5959

60-
public static final String NEOFORGE = "neoForge";
60+
public static final String FORGE_USERDEV = "forgeUserdev";
6161
public static final String NFRT_TOOL = "neoFormRuntimeTool";
6262

6363
public static final String MINECRAFT_COMPILE_LIBRARIES = "minecraftLibraries";

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

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -224,7 +224,7 @@ public File getMinecraftServerJar() {
224224

225225
@Override
226226
public File workingDir() {
227-
return neoForgeWorkingDirectory(configContext.project(), minecraftVersion(), neoForgeDependency());
227+
return forgeWorkingDirectory(configContext.project(), minecraftVersion(), forgeUserdevDependency());
228228
}
229229

230230
@Override
@@ -234,7 +234,7 @@ public ModPlatform getModPlatform() {
234234

235235
@Override
236236
public String getJarPrefix() {
237-
return "nfrt-" + mangleNeoForgeVersion(neoForgeDependency()) + "-";
237+
return "nfrt-" + mangleForgeVersion(forgeUserdevDependency()) + "-";
238238
}
239239

240240
@Override
@@ -247,10 +247,10 @@ public ForgeLaunchConfigs.Config getLaunchConfig() {
247247
}
248248

249249
protected @NonNull String neoForgeNotation() {
250-
return neoForgeDependency().getGroup() + ":" + neoForgeDependency().getName() + ":" + neoForgeDependency().getVersion();
250+
return forgeUserdevDependency().getGroup() + ":" + forgeUserdevDependency().getName() + ":" + forgeUserdevDependency().getVersion();
251251
}
252252

253-
protected ExternalModuleDependency neoForgeDependency() {
254-
return Objects.requireNonNull(Objects.requireNonNull(metadataProvider, "Metadata provider not setup").getNeoForgeDependency(), "NeoForge version not setup");
253+
protected ExternalModuleDependency forgeUserdevDependency() {
254+
return Objects.requireNonNull(Objects.requireNonNull(metadataProvider, "Metadata provider not setup").getForgeUserdevDependency(), "NeoForge version not setup");
255255
}
256256
}

0 commit comments

Comments
 (0)