Software-Noob / pterodactyl-images

Pterodactyl Docker images for Pelican/Pterodactyl/WISP Game Panels
MIT License
142 stars 46 forks source link

Fix Unable to change to /home/container #53

Closed Lefraudeur closed 1 year ago

Lefraudeur commented 1 year ago

"Unable to change to /home/container. Something went real wrong." was shown even if cd /home/container succeeded.

Software-Noob commented 1 year ago

Thanks for catching that! That was a silly mistake on my end as this ends up being an if-else one liner.

I prefer to group commands instead of executing them in a subshell. After further review, I don't even see why to echo anything - failure to cd into the bind mount would fail to start the container anyways.

I've removed the echo completely in https://github.com/Software-Noob/pterodactyl-images/commit/45f9944c5958b4e622843e9e3ec90a8659c37f82