MinecraftServerControl / mscs

Powerful command-line control for UNIX and Linux powered Minecraft servers
https://minecraftservercontrol.github.io
BSD 2-Clause "Simplified" License
485 stars 62 forks source link

Use Perl's switch parsing argument to pass variables to scripts #246

Closed sandain closed 4 years ago

sandain commented 4 years ago

This is in the same line as commit 7c4b4545c07e273a815135fc04c4d1ee3a140f1b.

sandain commented 4 years ago

I've been running this patch for a while now, but I'm not sure I have tested everything. Please look over the patch and let me know if you see any issues.

sandain commented 4 years ago

This is a more robust way of passing variables into Perl. I made this patch for two reasons, i) to use the same pattern as was done in commit 7c4b4545c07e273a815135fc04c4d1ee3a140f1b, that fixed an issue (#240) with parsing spaces in properties files, to simplify future code debugging, and ii) it actually fixes a bug I found while working on new code for msc-gui.

Roflicide commented 4 years ago

i see, very nice !