This fixes a minor issue with some scripts in bin/ that use the shebang line #!/usr/bin/env sh.
When running on a system with an actual sh implementation, these scripts fail due to a syntax issue with the function keyword (apparently a bash specific thing). This is easy to miss on platforms where sh is just a link to bash, so using bash will ensure that the scripts work without changes on all platforms.
Testing
Steps:
Run npm run docs to exercise all these scripts and ensure they are still functional.
Checklist
Make sure you check off the following items. If they cannot be completed, provide a reason.
mportiz08
commented
1 month ago
Bug/issue #, if applicable:
Summary
This fixes a minor issue with some scripts in
bin/that use the shebang line#!/usr/bin/env sh.When running on a system with an actual
shimplementation, these scripts fail due to a syntax issue with thefunctionkeyword (apparently abashspecific thing). This is easy to miss on platforms whereshis just a link tobash, so usingbashwill ensure that the scripts work without changes on all platforms.Testing
Steps:
npm run docsto exercise all these scripts and ensure they are still functional.Checklist
Make sure you check off the following items. If they cannot be completed, provide a reason.
npm test, and it succeeded