search

QuiltMC / quilt-installer

The Quilt installer for the official Minecraft launcher and servers
Apache License 2.0
40 stars 20 forks source link

Fix crash in the server installer on Windows #7

Closed EnnuiL closed 3 years ago

EnnuiL commented 3 years ago

Previously, if you attempted to install a Quilt server with the installer on Windows, it would say that it finished installing it in the GUI despite doing nothing and the CLI would show a more helpful error saying that a semicolon is an illegal character for the temporary library files. This PR fixes that by only using the artifact ID joining together the artifact ID and version and use it for the temporary file name

i509VCB commented 3 years ago

This does bring up a point, we probably should run a server install via cli to test in CI.