• Jason Rhinelander's avatar
    Make check-style.sh work on stock macOS · d080f833
    Jason Rhinelander authored
    ./tools/check-style.sh fails on stock OS X currently; this fixes it:
    
    - use pipes directly rather than exec redirection (macOS's ancient
      version of bash fails with the latter)
    - macOS's ancient bash doesn't support '\e' escapes in `echo -e`;
      replace with \033 instead
    - BSD grep doesn't support GREP_COLORS, but does allow GREP_COLOR.
      Adding both doesn't hurt GNU grep: GREP_COLOR is deprecated, and won't
      be used when GREP_COLORS is set.
    - BSD grep doesn't collapse multiple /'s in the listed filename, so
      failures under `include/` would should up as
      `include//pybind11/whatever.h`.  This removes the / from the include
      directory argument.
    
    Minor other changes:
    - The CRLF detection runs with -l, so GREP_COLORS wasn't doing
      anything; removed it.
    - The trailing whitespace test would trigger on CRLFs, but the CR would
      result in messed up output.  Changed the test to just match trailing
      spaces and tabs, rather than all whitespace.
    d080f833
check-style.sh 2.25 KB