Remove bashisms

[dertuxmalwieder]

In order to be more compatible with several operating systems (not all UNIX-like systems have a bash installed), it might be a good idea to get rid of the bashisms and set the shebang to /bin/sh instead. As far as I can see, there is no functionality that would need to be removed:

• echo -e does not seem to be necessary where it occurs -> remove the -e
• read -p would need to be rewritten as printf "some text" ; read -r
• $(...) - inconsequently used in this script anyway - should be replaced with backticks
• the [[ var == text ]] syntax only needs minor changes

The benefit from this would be that nobody would need to use a bash with all of its dependencies -> even lower entry point.

[duxsco]

fyi, regarding:

• $(...): https://mywiki.wooledge.org/BashFAQ/082
• set -euo pipefail: https://mywiki.wooledge.org/BashFAQ/105 => Dont' use them

[duxsco]

shellcheck should be used prior to publication, or the script should be declared as alpha.

[shombando]

Thanks @dertuxmalwieder and @duxsco, I'll investigate and work towards removing bash dependency.

[duxsco]

I don't mind bash dependency 🙂, but I can understand people wanting to see /bin/sh used if sufficient.

[dertuxmalwieder]

Thank you for the additions, @duxsco :)