search

BeardedManStudios / ForgeNetworkingRemastered

See various forks, also please join the Forge Community fork of Forge Alloy! -> https://github.com/ForgeAlloyCommunity/ForgeAlloy
https://twitter.com/FarrisFaulds
Apache License 2.0
1.49k stars 309 forks source link

Issue 277 - MasterServer params not working #280

Closed phalasz closed 5 years ago

phalasz commented 5 years ago

Change out the command line parser to a nuget package that provides a more robust argument parsing funtionality. Fixes issue #277

Using CommandLine Parser 1.9.17 (last version that uses .Net 3.5) to parse the options passed in from the command line. See https://github.com/gsscoder/commandline/tree/stable-1.9.71.2 for more information on the package.

Not sure if commiting the nuget package is the correct way to go. Please let me know if I should not commit the package and I'll get that removed.

BrentFarris commented 5 years ago

I have merged a big cleanup branch and so you will probably need to merge current develop into your fork and resolve conflicts, sorry about this. The main breaking change in develop is the folder paths have updated their names to not have spaces. Please see this commit for more info on the changes

BrentFarris commented 5 years ago

Thanks 👍 looks like they have the same license so we are good :)