From 3404521d5d06687273ad16dd0c27512efb8f3e2e Mon Sep 17 00:00:00 2001 From: Richard van der Hoff Date: Wed, 1 Feb 2017 16:11:11 +0000 Subject: [PATCH] Fix jenkins build Do `npm install` on js-sdk rather than `npm run build`, which will hopefully mean that `browserify` gets installed before we try to run it. The README says we should use `npm install` for the react-sdk too, so let's do that, and bring the travis and jenkins builds into sync with the README. --- .travis.yml | 2 +- scripts/jenkins.sh | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/.travis.yml b/.travis.yml index af738bb429..9720d8872f 100644 --- a/.travis.yml +++ b/.travis.yml @@ -4,4 +4,4 @@ node_js: install: - npm install - (cd node_modules/matrix-js-sdk && npm install) - - (cd node_modules/matrix-react-sdk && npm run build) + - (cd node_modules/matrix-react-sdk && npm install) diff --git a/scripts/jenkins.sh b/scripts/jenkins.sh index 5ccc199164..0f4fac2513 100755 --- a/scripts/jenkins.sh +++ b/scripts/jenkins.sh @@ -20,8 +20,8 @@ rm -r node_modules/olm cp -r olm/package node_modules/olm # we may be using dev branches of js-sdk and react-sdk, in which case we need to build them -(cd node_modules/matrix-js-sdk && npm run build) -(cd node_modules/matrix-react-sdk && npm run build) +(cd node_modules/matrix-js-sdk && npm install) +(cd node_modules/matrix-react-sdk && npm install) # run the mocha tests npm run test