Skip to content

Commit

Permalink
ci: Parallelize make install in CI (#2080)
Browse files Browse the repository at this point in the history
  • Loading branch information
mcous authored Sep 4, 2018
1 parent b32df5e commit 4c5b258
Show file tree
Hide file tree
Showing 3 changed files with 11 additions and 5 deletions.
2 changes: 1 addition & 1 deletion .travis.yml
Original file line number Diff line number Diff line change
Expand Up @@ -46,7 +46,7 @@ before_install:
- source scripts/travis-install.sh

install:
- make install
- make -j 2 install

before_script:
# Get all branches to allow comparison for mono-repo management tools
Expand Down
12 changes: 9 additions & 3 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -30,13 +30,19 @@ endif
usb_host = $(shell yarn run -s discovery find -i 169.254 fd00 -c "[fd00:0:cafe:fefe::1]")

# install all project dependencies
# front-end dependecies handled by yarn
.PHONY: install
install:
install: install-py install-js

.PHONY: install-py
install-py:
$(OT_PYTHON) -m pip install pipenv==11.6.8
$(MAKE) -C $(API_LIB_DIR) install
$(MAKE) -C $(API_DIR) install
$(MAKE) -C $(UPDATE_SERVER_DIR) install

# front-end dependecies handled by yarn
.PHONY: install-js
install-js:
yarn
$(MAKE) -C $(SHARED_DATA_DIR) build
$(MAKE) -C $(DISCOVERY_CLIENT_DIR)
Expand All @@ -45,7 +51,7 @@ install:
# TODO(mc, 2018-03-22): API uninstall via pipenv --rm in api/Makefile
.PHONY: uninstall
uninstall:
lerna clean
shx rm -rf '**/node_modules'

# install flow typed definitions for all JS projects that use flow
# typedefs are commited, so only needs to be run when we want to update
Expand Down
2 changes: 1 addition & 1 deletion appveyor.yml
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@ install:
- ps: $env:nodejs_version = (Get-Content -Path .nvmrc)[0]
- ps: Install-Product node $env:nodejs_version x64
# install dev dependencies
- cmd: '%MAKE% install'
- cmd: '%MAKE% -j 2 install'

# do not run MSBuild
build: false
Expand Down

0 comments on commit 4c5b258

Please sign in to comment.