Skip to content
Snippets Groups Projects

fix(Modloader installer): Correctly setup ProcessBuilder for Modloader Installer

Merged Griefed requested to merge issue-69-wrong-server-loader into main

The problem was a wrongly build ProcessBuilder not passing the users configuration to said ProcessBuilder correctly. Vital information about the Minecraft and Fabric Loader versions went poof due to me not setting up the ProcessBuilder in the way it was supposed to. This has been fixed and the Fabric server is now installing for the correct loader and Minecraft versions.

Before: I passed my arguments as String and formatted Strings.

Now: I'm passing a String List which contains all the vital information needed.

Merge request reports

Approved by

Merged by GriefedGriefed 3 years ago (Jul 21, 2021 5:41pm UTC)

Merge details

  • Changes merged into main with 12dec96d.
  • Deleted the source branch.

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading