search

SKCraft / Launcher

🚀 Distribute your Minecraft modpacks with a custom launcher
Other
620 stars 433 forks source link

Version Type set to the wrong value ? #507

Closed oxodao closed 1 year ago

oxodao commented 1 year ago

I was writing the Launching the game page on the wiki.vg the other day and I found that the versionType set in the game launch args seems to be wrong.

Here's where its found in the source: https://github.com/SKCraft/Launcher/blob/f45033e4fa81534a75c2802ec1116afe7f6df8cd/launcher/src/main/java/com/skcraft/launcher/launch/Runner.java#L440

I am not doing a PR since I'm not sure what is the real value that should be put here but I'd like to gather info if someone knows about it.

MultiMC seems to be setting it to snapshot/release, I'm guessing the value set by SKCraft launcher is there because it's a modpack launcher but it could also just be a misunderstanding / an oversight

What's the deal with this variable ?

hedgehog1029 commented 1 year ago

The vanilla launcher sets it to "release" or "snapshot", but it's basically unused in game code - only places it's used are for the version string on the main menu and in the debug menu.

(More specifically, as far as I know, the vanilla launcher just reads the "type" key from the version manifest, and uses that. For example in the 1.19 manifest it's set to "release")

oxodao commented 1 year ago

Ok, so not that important. Thanks !