Set the ConfigFormat to use quotes around all settings. I believe this was causing games with spaces in the name to be non-joinable. The strange part is you could set it back to having a space and it would work going forward.
Also set the ExitMethod to String and quit. The user must type yes following since there is no option to do two input commands to quit a server. CtrlC doesn't work in AMP on Windows for this game. And CtrlC doesn't work for this game on the Linux Terminal. OS_CLOSE ended in a kill.
Removed the config file and used the start/wait/stop update stages instead.
During the Proton feature enhancement I accidentally removed the Steam SDK update stage. This should resolve it so players can view their server in the browser again.
Based on Greelan's review, I've updated the following:
1. Removed unnecessary ports
2. Clarified Max Heap needs "m" at the end
3. Added "" around startup args that were manually inserted
This switches the config to use a base config rather than a template to allow users to manually add special settings without a full config file overwrite.
This should be the final version of the config, ready for release. The only issue is there is no "shutdown" command for this game without using the GUI. The only way to stop it within AMP (as far as I can figure out) is to do a force stop.
Notes by Greelan:
Some further development of the template that you can merge if appropriate to build out your PR.
Having worked with it a bit more I've concluded that it ain't a particularly AMP-friendly server.
Couple of specific comments:
I've added AdminPassword as a command line argument, but I am not sure that is accepted. Most instructions say to put it in the ServerSettings.ini. Problem is that the default ServerSettings.ini that is created when the server is first started does not include that field. So I tried the solution of having a copy of that ini, with the AdminPassword field added, being downloaded from GitHub and copied across as update stages. But the server overwrites that on first start. And given I don't want the update stages overwriting the existing ServerSettings.ini, because it might include a lot of user configuration, I can't see a resolution for this (ie getting an AdminPassword entry into the standard ServerSettings.ini that is not overwritten by the server and that does not cause AMP to overwrite other settings).
The Server Password manifest entry assumes that it will work with the ServerPassword field that exists by default in ServerSettings.ini. But I've also read that ServerPassword only works in Engine.ini. However, Engine.ini doesn't include that entry by default - which creates the same issue as for AdminPassword.
This update allows Risk of Rain 2 to run under Proton and xvfb. This requires the user to run the command below to install prerequisites.
apt-get install xvfb lib32gcc1
- Use OS_CLOSE for saner shutdown (on Linux at least)
- Add start/wait/stop update stages
- Shorten exit timeout (for Windows)
- Other tweaks (RCON port, command line args, manifest)
Changed how passwords are handled for IS Sever. Fixed Password Field to be hidden for support tab, and Updated Server Config Page to allow for new configuration options. Will add support for game.ini at a later date.
Added Insurgency Sandstorm support to AMP. Support for Windows and Linux, Game INI configuration to be added as a web setting at a later date, as well as cleanup of REGEX.
