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.

https://codereview.appspot.com/569540043/

Reply via email to