Commit d5c4e9e1 authored by Dylan Trotter's avatar Dylan Trotter Committed by GitHub

Clean up travis config, set up os x build (#143)

* Remove dependence on pip.
* Add osx to Travis test matrix.
* Remove pip upgrade and sudo requirement for Travis config.
parent 3dc695ec
language: go language: go
# Upgrade default Travis pip which is old and doesn't work the way we expect. os:
# Need sudo to do that. - linux
sudo: required - osx
install: pip install --upgrade pip
# Run gofmt and lint serially to avoid confusing output. Run tests in parallel # Run gofmt and lint serially to avoid confusing output. Run tests in parallel
# for speed. # for speed.
script: make gofmt lint; make -j2 test script: make gofmt lint && make -j2 test
...@@ -159,13 +159,15 @@ gofmt: build/gofmt.diff ...@@ -159,13 +159,15 @@ gofmt: build/gofmt.diff
$(GOLINT_BIN): $(GOLINT_BIN):
@go get -u github.com/golang/lint/golint @go get -u github.com/golang/lint/golint
golint: $(GOLINT_BIN) $(PYLINT_BIN) golint: $(GOLINT_BIN)
@$(GOLINT_BIN) -set_exit_status runtime @$(GOLINT_BIN) -set_exit_status runtime
# Old versions of pip don't support the --prefix param so specify the bin dir # Don't use system pip for this since behavior varies a lot between versions.
# (--install-scripts) and the Python dir (--target) separately. # Instead build pylint from source. Dependencies will be fetched by distutils.
$(PYLINT_BIN): $(PYLINT_BIN):
@pip install --install-option=--install-scripts='$(ROOT_DIR)/build/bin' --target '$(PY_DIR)' pylint @mkdir -p build/third_party
@cd build/third_party && curl -sL https://pypi.io/packages/source/p/pylint/pylint-1.6.4.tar.gz | tar -zx
@cd build/third_party/pylint-1.6.4 && $(PYTHON) setup.py install --prefix $(ROOT_DIR)/build
pylint: $(PYLINT_BIN) pylint: $(PYLINT_BIN)
@$(PYLINT_BIN) compiler/*.py $(addprefix tools/,benchcmp coverparse diffrange grumpc grumprun) @$(PYLINT_BIN) compiler/*.py $(addprefix tools/,benchcmp coverparse diffrange grumpc grumprun)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment