On 2020/03/14 21:58:52, lemzwerg wrote: > Just skimming the code: LGTM > > https://codereview.appspot.com/569540043/diff/555440061/scripts/build/output-distance.py > File scripts/build/output-distance.py (right): > > https://codereview.appspot.com/569540043/diff/555440061/scripts/build/output-distance.py#newcode66 > scripts/build/output-distance.py:66: # explicitly use bash, so we don't get dash > on Ubuntu. > This needs an update of `STEPMAKE_INIT` in `aclocal.m4`. Right now, AFAICS, > `bash` is optional, and `configure` falls back to whatever is given in the > `SHELL` environment variable.
Are there still systems that run without bash that we care about? I created http://codereview.appspot.com/579480047 for looking more closely at how we configure shell. https://codereview.appspot.com/569540043/
