[WIP] remove shopt&alias to make it work on powershell

[pvgenuchten]

PR to facilitate discussion on this topic

docker-compose was removed >1 year ago, i think we can assume all are migrated by now?

removing shopt and alias because it is not supported on powershell/windows

todo: update docs for windows:
syntax: bash geopython-workshop-ctl.sh start|stop|url|update

• works on powershell (windows+docker desktop)
• works on ubuntu (WSL)

[pvgenuchten]

@tomkralidis, updated docs for powershell

[justb4]

Hm, looks like these changes remove the alias and assume the command is docker compose. This will break for students having docker-compose installed (like me).

Best option is to make the script cross-platform. The $IsWindows var e.g.
and https://redmondmag.com/articles/2020/08/18/cross-platform-powershell-scripts.aspx

[pvgenuchten]

ah, good to know people are still using docker-compose

alternative would be to duplicate the script and rename to _ps.sh

@justb4 PR updated

[justb4]

is this still required? I guess the openapp="cmd /c start" is the command in PS here.

[justb4]

I cannot test this, but seems useful. I wonder if the uname stuff even works. I have no idea how Powershell relates to WSL and Docker install. I have the feeling we would not need a separate Bash script with some smart 'sniffing' like the $Windows var and uname command, branching in the original script. It may be confusing even, two scripts with almost identical names.