Permalink
Browse files

get bashate running on OSX

  • Loading branch information...
austin987 committed Jun 18, 2017
1 parent 8d413b4 commit 5060f652dcb2c9b1765e049f6f56d9ff1a2f9086
Showing with 4 additions and 7 deletions.
  1. +2 −2 .travis.yml
  2. +2 −5 tests/shell-checks
View
@@ -29,8 +29,8 @@ before_install:
- if [[ "$TRAVIS_OS_NAME" == "linux" ]] ; then time sudo apt-get install python-bashate shellcheck ; fi
- if [[ "$TRAVIS_OS_NAME" == "osx" ]] ; then time brew update ; fi
# No bashate on homebrew yet
#- if [[ "$TRAVIS_OS_NAME" == "osx" ]] ; then time brew install checkbashisms bashate shellcheck ; fi
# Bashate isn't on homebrew, they don't package things that are on pypi, so we have to pip install it instead:
- if [[ "$TRAVIS_OS_NAME" == "osx" ]] ; then time pip install bashate ; fi
- if [[ "$TRAVIS_OS_NAME" == "osx" ]] ; then time brew install checkbashisms shellcheck ; fi
# Note if testing on a branch, you can replace this with your desired command, e.g.,:
View
@@ -66,10 +66,7 @@ do
# Can't ignore individual things for now, filed bug:
# https://bugs.launchpad.net/bash8/+bug/1698088
# E003=indent spacing check
# E006=line length check
# E010=do/while same line (in some perl in winetricks)
# Also, not on homebrew yet, so don't expect it to always be around:
if [ -x "$(which bashate)" ] ; then
w_try bashate -i E003,E010 "${shellscript}"
fi
w_try bashate -i E003,E006,E010 "${shellscript}"
done

0 comments on commit 5060f65

Please sign in to comment.