Skip to content

Create config for Wurm Unlimited#44

Merged
own3mall merged 15 commits intoOpenGamePanel:masterfrom
Raz89x:patch-1
Mar 2, 2017
Merged

Create config for Wurm Unlimited#44
own3mall merged 15 commits intoOpenGamePanel:masterfrom
Raz89x:patch-1

Conversation

@Raz89x
Copy link
Contributor

@Raz89x Raz89x commented Feb 21, 2017

Config xml for Wurm Unlimited

Copy link
Contributor

@Zorrototo Zorrototo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I 'reviewed' your xml thinking it was just comments placed on different parts of the file, not sure what it is actually as it now shows as 'pending' lol. (edit: after finishing and sending from this discussion page, it shows as I though)

<game_key>wurmu_win64</game_key>
<installer>steamcmd</installer>
<game_name>Wurm Unlimited Server Windows</game_name>
<server_exec_name>WurmServerLauncher</server_exec_name>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it may work like this but it needs the .exe

<cli_params>
<cli_param id="IP" cli_string="ip=" />
<cli_param id="PORT" cli_string="externalport=" />
<cli_param id="QUERY_PORT" cli_string="queryport=27016" />
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You should not set a port like that. What if people install multiple of this server? they all use the port 27016? you should use the reserve_port thingy to set the query port and add +1 to the game port, this way it will always use game port+1 for query port this way.

<server_params>
<param id="SERVERNAME" key="servername=" type="text">
<option>ns</option>
<desc>The name of the server that will be visible in server browsers. (NOTE: If the server name contains spaces, quotes are required! ie. "Server Name")</desc>
Copy link
Contributor

@Zorrototo Zorrototo Feb 21, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you can use different options to do what you want, I think it is:

    nothing / no value = placed as is
    s = space / separated
    q = quoted
    sq = space and quotes
    sc = space and ends with a comma
    sqc = space, quoted, and ends with a comma

You can't trust people to do things right :D, make them do it right without any extra effort.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

True but i tried various options and i couldn't get it to work correctly, the cli param needs to have quotes if it has a space but also it cant have a separating space ie (servername="server Name")

this was the only way i could get it to work, i might have overlooked something though...

@rocco27
Copy link
Contributor

rocco27 commented Feb 21, 2017

4 more things to make your xml nicer:

  • Please do not set custom game name, follow the original game name. Example: Wurm Unlimited Server Windows is total wrong and unnecessery, use only Wurm, it is enought.
  • You can set the hostname automaticly and we do this usually. The owner always set the hostname in the OGP, why we force them to do it twice?
  • Same with the admin password, this called RCOn generally and OGP can manage it
  • Could not find a correct query type? Why is it missing?

Anyway it's nice, thanks for your work!

@own3mall
Copy link
Contributor

Please make the changes recommended above, update your branch and pull request, and then I'll merge it. Thanks!

smalls89 added 4 commits February 22, 2017 02:19
default game port is 3724, add 23292 for queryport 27016
@Raz89x
Copy link
Contributor Author

Raz89x commented Feb 22, 2017

rocco

Same with the admin password, this called RCOn generally and OGP can manage it
Could not find a correct query type? Why is it missing?

Im assuming these are refering to rcon in the way most games use it... Wurm does not have such a feature

the admin password is to be used ingame via the console

else there is the prompt window of the server, but you can't type in it...

@rocco27
Copy link
Contributor

rocco27 commented Feb 22, 2017

Doesn't matter how can you manage RCON/Admin ingame, when I said OGP can manage it I thought you can set the password automaticly in the start up command line by OGP. If the game server support general RCON then it is other thing and maybe you can manage the RCON comands in the Log window from console

@Raz89x
Copy link
Contributor Author

Raz89x commented Feb 22, 2017

I imagine these ports can be set to anything, but these are defaults

Wurm is unique when i comes to admin, everything is done through an in game item oe in game console, there is NO external console prompt, rcon, or otherwise

thanks rocco for pointing out the queryport

also found vac setting
@Zorrototo
Copy link
Contributor

Just a question, why do you have this post install? is it mandatory when you install a fresh server? The post install script is used after the Install or update of server.

@Raz89x
Copy link
Contributor Author

Raz89x commented Feb 23, 2017

I saw no alternative as the install puts down the 2 backup maps. If the user chooses to use the default maps, the server will throw the biggest fit about the maps not being found.

Side note with using q option, CLI separates it with a space, don't know if this is a bug/glitch, orI'm just doing it wrong...

@Zorrototo
Copy link
Contributor

Zorrototo commented Feb 24, 2017

You mean the "q" option for servername in server parameters is not working for you? be more specific to speed up the polishing of your xml :)

edit: ok i added the "q" option to servername parameter, and it adds quotes but it also adds a space BEFORE the quoted servername. I think it is a bug yes, because it shouldn't. Panel and Agent on latest version for me.
edit5: I tested with <option></option> with <option>q</option> and with <option>sq</option> and the result is that it always add space and quotes around the servername resulting in command line: servername= "test 1 2 3" and it fails to set the servername.

Also, I'm thinking about the post_install, and for what I understand, it will overwrite these folder every update, and it might be a problem as I see these folders are the ones containing configuration files. Tell me if I'm right but if it is the case, you should have some kind of fail-safe, if the folders already exist you shouldn't move the "_backup" ones to avoid overwriting config files.

edit2: hu, it should be ok actually, mv command shouldn't overwrite by default

edit3: something I noticed, install, it moves "Adventure_backup" to "Adventure". Delete steamapps folder in this game server root (this will force validation on file and redownload missing/corrupted files), run update again, it moves "Adventure_backup" INSIDE "Adventure", in a subfolder called "Adventure" (so Adventure_backup/ becomes Adventure/Adventure/). Delete steamapps again, update, it then tells you:

mv: cannot move ‘./Adventure_backup’ to ‘./Adventure/Adventure_backup’: Directory not empty
mv: cannot move ‘./Creative_backup’ to ‘./Creative/Creative_backup’: Directory not empty

It seems mv command first simply 'rename' "Adventure_backup" to "Adventure" because there is no "Adventure" folder yet.
Then because "Adventure" folder exists, it actually moves "Adventure_backup" inside the "Adventure" folder.
Then it tells you it cannot move "Adventure_backup" because files already exist in destination.

This will not really create any problem but just wanted to mention it because after multiple update you will face this behavior.

edit4: also your VAC parameter seems to be not valid for Wurm Unlimited absolutely no reference in official doc, and even the log tells you it is wrong.

@own3mall
Copy link
Contributor

If you look at #19 there is only ns... adding new options now.

own3mall added a commit that referenced this pull request Feb 24, 2017
@own3mall
Copy link
Contributor

Hope I didn't break anything. I documented the changes:

https://github.com/OpenGamePanel/OGP-Website/wiki/XML-Notes

If any of you want to help complete the XML documentation following a format similar to the one in place (unless you can come up with a better one), I'll grant you access. We can add to it as confusion and clarification is needed.

smalls89 added 2 commits February 24, 2017 03:12
found this to be a little neater and more effective
vac
odd, seems i was having a glitch... my server showed up without vac until i put -secure in it.... seems to have gone away
@Raz89x
Copy link
Contributor Author

Raz89x commented Feb 24, 2017

so option q should work now???

smalls89 added 3 commits February 24, 2017 03:57
found my own answer
RMI port and RMI registration port move to reserved
query port lowered to +3
seems empty defaults unchecked...
@Zorrototo
Copy link
Contributor

as I said

edit4: also your VAC parameter seems to be not valid for Wurm Unlimited absolutely no reference in official doc, and even the log tells you it is wrong.

Unknown parameter: -insecure

@own3mall
Copy link
Contributor

Yes, "q" should work.

@Zorrototo
Copy link
Contributor

@own3mall if you can create a sub page in the wiki like "WIKI WIP" where I can import the current one and start complete it, test things and all (I'm new to GitHub and don't want to break anything on the real one :D) and grant me access to this sub-page, I'll start doing it. When I'll have it complete, polished and verified we can then import it in the real one and destroy this 'WIKI WIP" page.

@Raz89x
Copy link
Contributor Author

Raz89x commented Feb 25, 2017

I suppose this is done.... Unless there is a way to edit sqlite, all the remaining settings are in a sqlite db within the respective map folder...

@Zorrototo
Copy link
Contributor

Zorrototo commented Feb 25, 2017

Seems pretty good yes. Don't have the game to test but server starts good.

Can you change it back to win32 please, this way we keep the good organisation for ogp xml :D (some older xml don't but for now better use the good naming convention) edit: win32 can run on both 32 and 64 bit when win64 is restricted to 64 bits OS.

@Zorrototo
Copy link
Contributor

Zorrototo commented Feb 26, 2017

Something you can also add is an icon for this game server, I think after that this can be accepted as a functional XML :) 👍

edit: like this one https://www.dropbox.com/s/lnkbbcxrhrcatf4/wurmu.png?dl=0

@Raz89x
Copy link
Contributor Author

Raz89x commented Feb 27, 2017

ih, i thought the win32 and win64 where for specific versions, wurm is indifferent, it works on either, and depends on the version of java installed locally, can change it back if it is still an issue...

@Raz89x
Copy link
Contributor Author

Raz89x commented Feb 27, 2017

idk how the icons work....

@Zorrototo
Copy link
Contributor

For your XML simply upload the icon to OGP Panel folder /images/icons/ name should match XML so wurmu.png will work.

@own3mall
Copy link
Contributor

Yay for making progress! Let me know when this is ready.

@Raz89x
Copy link
Contributor Author

Raz89x commented Feb 28, 2017

believe thats about it...

@own3mall own3mall merged commit f1f6c68 into OpenGamePanel:master Mar 2, 2017
@Raz89x Raz89x deleted the patch-1 branch March 3, 2017 00:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

Comments