Skip to content

Update monitor_gsquery.sh#884

Merged
dgibbs64 merged 1 commit intoGameServerManagers:masterfrom
igeighty:patch-1
Jun 14, 2016
Merged

Update monitor_gsquery.sh#884
dgibbs64 merged 1 commit intoGameServerManagers:masterfrom
igeighty:patch-1

Conversation

@igeighty
Copy link
Copy Markdown
Contributor

this fixes the monitor function for arma3 as @MPCP10 suggested

this fixes the monitor function for arma3 as @MPCP10 suggested
@UltimateByte
Copy link
Copy Markdown
Contributor

So it's tested and working as you said ?

@igeighty
Copy link
Copy Markdown
Contributor Author

Yes, I am using the default port value (2302) specified in line 35 of arma3server file (i still have the ports commented out in arma3-server.server.cfg) as these are not needed if you are specifying the port in the startup parameters.

the ./arma3server m now queries 2303 correctly. as @MCPC10 says, it is the base port +1 that needed to be read.

so working good now with the added lines. thank you.

@UltimateByte
Copy link
Copy Markdown
Contributor

@dgibbs64 will review this and probably merge :)

@dgibbs64 dgibbs64 merged commit cef1abe into GameServerManagers:master Jun 14, 2016
@dgibbs64
Copy link
Copy Markdown
Member

Good work 😁

@igeighty
Copy link
Copy Markdown
Contributor Author

sorry i seem to have messed up the file edit that i uploaded..

i spelt the engine name incorrectly.. real virtuality i have double and triple checked it now, and will submit a new pull.

so sorry guys. i typed it manually and made the typo..

@JHulsmans
Copy link
Copy Markdown

Hey @igeighty happens to the best of us! Thanks for your contribution (again)

@igeighty
Copy link
Copy Markdown
Contributor Author

no worries guys, first go at uploading and i bust it :)

@lock
Copy link
Copy Markdown

lock Bot commented Jul 19, 2018

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock Bot locked as resolved and limited conversation to collaborators Jul 19, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants