[GH-ISSUE #534] Escaped Directory names #431

New issue

Closed

opened 2026-02-27 02:01:25 +03:00 by kerem · 4 comments

kerem commented

2026-02-27 02:01:25 +03:00

Owner

Originally created by @JimTR on GitHub (Nov 17, 2015).
Original GitHub issue: https://github.com/GameServerManagers/LinuxGSM/issues/534

Originally assigned to: @dgibbs64 on GitHub.

I have a problem with certain directory paths (i.e ones with spaces in) lgsm does not escape the spaces so the script tries to use a non valid path for its data from this line
rootdir="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
Example
returned rootdir = /home/arma/games/Arma 3 (invalid path)
actual rootdir = /home/arma/games/Arma\ 3
I know this is no biggy but it may be worth putting in the install documentation that using a directory with a space in it does not work as expected

Originally created by @JimTR on GitHub (Nov 17, 2015). Original GitHub issue: https://github.com/GameServerManagers/LinuxGSM/issues/534 Originally assigned to: @dgibbs64 on GitHub. I have a problem with certain directory paths (i.e ones with spaces in) lgsm does not escape the spaces so the script tries to use a non valid path for its data from this line rootdir="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)" Example returned rootdir = /home/arma/games/Arma 3 (invalid path) actual rootdir = /home/arma/games/Arma\ 3 I know this is no biggy but it may be worth putting in the install documentation that using a directory with a space in it does not work as expected

kerem

2026-02-27 02:01:25 +03:00

closed this issue
added the
type: bug
label

kerem commented

2026-02-27 02:01:25 +03:00

Author

Owner

@dgibbs64 commented on GitHub (Dec 16, 2015):

A recent change may of resolved this issue.

rootdir="$(dirname $(readlink -f "${BASH_SOURCE[0]}"))"

@jaredballou may be able to confirm this as he made the alteration.

@dgibbs64 commented on GitHub (Dec 16, 2015): A recent change may of resolved this issue. rootdir="$(dirname $(readlink -f "${BASH_SOURCE[0]}"))" @jaredballou may be able to confirm this as he made the alteration.

kerem commented

2026-02-27 02:01:25 +03:00

Author

Owner

@dgibbs64 commented on GitHub (Dec 19, 2015):

Need to test this then close

@dgibbs64 commented on GitHub (Dec 19, 2015): Need to test this then close

kerem commented

2026-02-27 02:01:25 +03:00

Author

Owner

@dgibbs64 commented on GitHub (Dec 20, 2015):

Completed various tests. Hopfully the changes will stop this issue. However let me know if you spot anything else

github.com/dgibbs64/linuxgsm@2e3c4fe18f

github.com/dgibbs64/linuxgsm@dfbcee27eb

github.com/dgibbs64/linuxgsm@ff1a3e7a8e

@dgibbs64 commented on GitHub (Dec 20, 2015): Completed various tests. Hopfully the changes will stop this issue. However let me know if you spot anything else https://github.com/dgibbs64/linuxgsm/commit/2e3c4fe18f43e595d2d59299099fe7242373f35d https://github.com/dgibbs64/linuxgsm/commit/dfbcee27ebba6cd820d9fb9916a945ac67f4428c https://github.com/dgibbs64/linuxgsm/commit/ff1a3e7a8eaa50b0bf47592d2991263166141831

kerem commented

2026-02-27 02:01:25 +03:00

Author

Owner

@lock[bot] commented on GitHub (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[bot] commented on GitHub (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.