CloudFormation Backup Machine specifies /bin/sh in package.sh

[Michael Palmer-Gee]

CloudFormation Backup Machine specifies /bin/sh in package.sh but the script isn't fully POSIX compliant (e.g. use of [[ ]]). This is fine in most cases because /bin/sh is often symbolically linked to bash. But in cases where /bin/sh is linked to something else (like later versions of Ubuntu which link to dash), this causes the script to crash. This link provides some good background: https://wiki.ubuntu.com/DashAsBinSh.

My suggestion is that the Atlassian team should either, change the script to be POSIX compliant, or explicitly link to bash... which causes its own problems but, at least, would make the script fall over earlier and be more obvious to fix.

Thoughts?

Mike