diff --git a/launcher-builder/src/main/java/com/skcraft/launcher/builder/loaders/ModernForgeLoaderProcessor.java b/launcher-builder/src/main/java/com/skcraft/launcher/builder/loaders/ModernForgeLoaderProcessor.java index 0814bee..cd8648b 100644 --- a/launcher-builder/src/main/java/com/skcraft/launcher/builder/loaders/ModernForgeLoaderProcessor.java +++ b/launcher-builder/src/main/java/com/skcraft/launcher/builder/loaders/ModernForgeLoaderProcessor.java @@ -47,8 +47,8 @@ VersionInfo.class); VersionManifest version = manifest.getVersionManifest(); - if (version.getId() != null) { - loaderName = version.getId(); + if (info.getId() != null) { + loaderName = info.getId(); } // Copy game arguments @@ -89,6 +89,13 @@ ModernForgeInstallProfile profile = mapper.readValue(data, ModernForgeInstallProfile.class); + if (!profile.getMinecraft().equals(manifest.getGameVersion())) { + // TODO: Ideally this would show up as a Problem when running the check command. + // Getting the data from here to there is quite difficult, however. + log.warning(String.format("The Forge installer inside loaders/ is for Minecraft version %s; your " + + "manifest is set to %s.", profile.getMinecraft(), manifest.getGameVersion())); + } + // Import the libraries for the installer result.getProcessorLibraries().addAll(profile.getLibraries()); diff --git a/launcher/src/main/java/com/skcraft/launcher/model/loader/profiles/ModernForgeInstallProfile.java b/launcher/src/main/java/com/skcraft/launcher/model/loader/profiles/ModernForgeInstallProfile.java index 93594af..da79aca 100644 --- a/launcher/src/main/java/com/skcraft/launcher/model/loader/profiles/ModernForgeInstallProfile.java +++ b/launcher/src/main/java/com/skcraft/launcher/model/loader/profiles/ModernForgeInstallProfile.java @@ -24,6 +24,7 @@ private List libraries; private List processors; private Map> data; + private String minecraft; public List toProcessorEntries(final String loaderName) { return Lists.transform(getProcessors(), new Function() {