Skip to content

Use /bin/sh instead of /bin/bash#47

Open
mkhon wants to merge 1 commit intopg-monz:masterfrom
mkhon:sh
Open

Use /bin/sh instead of /bin/bash#47
mkhon wants to merge 1 commit intopg-monz:masterfrom
mkhon:sh

Conversation

@mkhon
Copy link

@mkhon mkhon commented Oct 26, 2018

Alpine Linux docker images do not have /bin/bash.
/bin/sh (which is usually a Korn Shell) is enough for pg_monz scripts.

@pengbo0328
Copy link
Contributor

pg_monz needs /bin/bash environment and doesn't support /bin/sh.

@mkhon
Copy link
Author

mkhon commented Nov 2, 2018

I believe I fixed all the places where bash'isms were used so the scripts can now be used as is on

  • Alpine Linux
  • Debian/Ubuntu minimal (without bash)
  • *BSD (FreeBSD, NetBSD, OpenBSD)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants