exampe openbsd. You can view, comment on, or merge this pull request online at:
https://github.com/geany/geany/pull/1574
-- Commit Summary --
* bash may not found in the system
-- File Changes --
M tests/ctags/runner.sh (2)
-- Patch Links --
https://github.com/geany/geany/pull/1574.patch https://github.com/geany/geany/pull/1574.diff
@kloun would you be able to run the script through [checkbashisms](http://openports.se/sysutils/checkbashisms) to verify that it doesn't contain any Bash-specific features?
I think on some Linuxes `/bin/sh` refers to Bash/Dash in some kind of POSIX compatibility mode, needs testing there too.
no bashisms.
me@gentoo ~ $ checkbashisms runner.sh me@gentoo ~ $
Works for me in Ubuntu, and script runs with GNU bash with `--posix` too.
b4n approved this pull request.
Looks reasonable and I can't see any bashisms in there indeed (makes me wonder why I made it a Bash script in the first place, I guess the script probably grew out of it). And anyway if Travis, you guys and me test it OK, it should be safe.
Merged #1574.
github-comments@lists.geany.org