Linuxgsm: syntax error: operand expected (error token is "-")

[Tealk]

Hi, if i restart my Server i get this error:

[ .... ] Stopping ark-server: Arc
[ .... ] Stopping ark-server: Arc
[ .... ] Stopping ark-server: Process still bound. Awaiting graceful exit: 0
[ .... ] Stopping ark-server: Process still bound. Awaiting graceful exit: 1
[ .... ] Stopping ark-server: Process still bound. Awaiting graceful exit: 2/home/steam/lgsm/functions/command_stop.sh: line 183: let: -: syntax error: operand expected (error token is "-")
/home/steam/lgsm/functions/command_stop.sh: line 185: [[: -: syntax error: operand expected (error token is "-")


[  OK  ] Stopping ark-server: Arc

[ .... ] Log Manager ark-server: Starting
[  OK  ] Log Manager ark-server: Starting

[ INFO ] Log Manager ark-server: Removing logs older than 7 days

[  OK  ] Log Manager ark-server: Removed 2 log files

[ .... ] Starting ark-server: Arc
[  OK  ] Starting ark-server: Arc

[UltimateByte]

@cedarlug ?
https://github.com/dgibbs64/linuxgsm/blob/master/lgsm/functions/command_stop.sh#L185

[cedarlug]

Thanks for the head's up. Checking.

[cedarlug]

The good news is that the issue is benign. The message does not effect the restart behavior.

Pull request #909 will solve the issue and squash the error in the event the process id vanishes when queried (which is a GOOD thing).

[cedarlug]

Closed with #909 integration.

[UltimateByte]

run
./arkserver uf
to get the update ;)

[Tealk]

@UltimateByte thanks, i have already added the updated part manually

[kbgvirus]

Hi, Please note this issue is still happening as the command_stop.sh hasnt been updated with pid=${pid//[!0-9]/}

[UltimateByte]

@kbgvirus
What ?
Is this not enough ?
https://github.com/GameServerManagers/LinuxGSM/pull/909/files

./arkserver uf is required to get the update.

[kbgvirus]

@UltimateByte i have added the line to the script and now getting another error, i will comment the error as i receive it.

[UltimateByte]

Don't do that, run the update-functiions command as asked. Since it's another error, and this one is probably solved, i'm re-closing this.
I dont know if you're trolling or what, but this issue is solved.

[kbgvirus]

@UltimateByte I am not trolling, if you read my first comment that new line had not been added to the stop script and therefore the error was still there

[UltimateByte]

Well if you read the commit 909, you can see that the line has been added to command_stop.sh.
So that's why i don't get why you're still saying that.
Once again : https://github.com/GameServerManagers/LinuxGSM/pull/909/files

Edit : Been merged to Development branche, that's why you're saying that. lol.

Awaiting for merge to master then.

[kbgvirus]

Thank you, there is no reason to troll a good cause

[UltimateByte]

Sorry about my lack of trust, i don't know you & i assumed it had been merged to master directly.
Well, let's put it to under dev until merge.
And yeah, you can definitely add this manually until the merge then.

[kbgvirus]

Thank You

[dgibbs64]

@cedarlug I dont know why but im not sure this went in correctly. Please can you check the master repo and fix if missing. Thank you

[dgibbs64]

@cedarlug https://github.com/GameServerManagers/LinuxGSM/commit/b8c82acbe8f2aa99cfd669b54d8c3ec274aedea3

[cedarlug]

@kbgvirus - please update functions and verify that commit https://github.com/GameServerManagers/LinuxGSM/commit/b8c82acbe8f2aa99cfd669b54d8c3ec274aedea3 cleans up the issue.

[lock[bot]]

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.