fix(update-lgsm): incorrect wdir sometimes

[jusito]

Description

If lgsm updates itself it cds into functions directory but not back, this is in at least one case an issue.

Fixes #[issue] Discord LGSM cannot update itself, breaks when checking lgsm module
Got the same error some time ago as well: jusito/docker-ttt#9

In the "files changed" it looks weird, basically I just wrapped the cd into ( cd ".."; do somethin), this ensures that even if it exits somewhere in between it will cd back if outside of the block.

Example to trigger the issue and also the validation that the pr will fix it:

Check the log before and after "a server is already installed" for the first log is shows function directory.

#!/bin/bash

set -euo pipefail

function stop_after_update() {
	pid="$1"
	while ! grep -F 'Checking Dependencies' gmodserver.log; do
		sleep 1s
	done
	kill "$pid"
	while ps -p "$pid" -o pid= > /dev/null 2>&1; do
		sleep 1s
	done
	if [ -d "./functions/lgsm/" ]; then
		echo "Issue: found folder $(pwd)/functions/lgsm"
	else
		echo "Solved: no folder $(pwd)/functions/lgsm"
	fi
}

(
	cd "$(dirname "$0")"
	if [ -d bugreport ]; then
		echo "please remove folder \"$(pwd)/bugreport\""
		exit 1
	fi
	mkdir bugreport
	(
		cd bugreport
		wget -O linuxgsm.sh "https://github.com/raw/GameServerManagers/LinuxGSM/v23.1.0/linuxgsm.sh"
		chmod +x linuxgsm.sh
		(
			export LGSM_GITHUBUSER="GameServerManagers"
			export LGSM_GITHUBBRANCH="master"
			./linuxgsm.sh gmodserver
			sed -i 's/"v23.1.0"/"v21.5.1"/' ./gmodserver
			./gmodserver auto-install
		) | tee /dev/tty > gmodserver.log 2>&1 &
		stop_after_update "$!"
		mv gmodserver.log gmodserver.before.log

		rm -rf ../bugreport/lgsm

		(
			export LGSM_GITHUBUSER="jusito"
			export LGSM_GITHUBBRANCH="fix/update-lgsm"
			sed -i 's/"v23.1.0"/"v21.5.1"/' ./gmodserver
			./gmodserver auto-install
		) | tee /dev/tty > gmodserver.log 2>&1 &
		stop_after_update "$!"
		mv gmodserver.log gmodserver.after.log
	)
)

Type of change

• Bug fix (a change which fixes an issue).
• New feature (a change which adds functionality).
• New Server (new server added).
• Refactor (restructures existing code).
• Comment update (typo, spelling, explanation, examples, etc).

Checklist

PR will not be merged until all steps are complete.

• This pull request links to an issue.
• This pull request uses the develop branch as its base.
• This pull request subject follows the Conventional Commits standard.
• This code follows the style guidelines of this project.
• I have performed a self-review of my code.
• I have checked that this code is commented where required.
• I have provided a detailed enough description of this PR.
• I have checked if documentation needs updating.

Documentation

If documentation does need updating either update it by creating a PR (preferred) or request a documentation update.

• User docs: https://github.com/GameServerManagers/LinuxGSM-Docs
• Dev docs: https://github.com/GameServerManagers/LinuxGSM-Dev-Docs

Thank you for your Pull Request!

[github-actions]

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