mirror of
https://github.com/appium/appium.git
synced 2026-10-28 04:14:00 +08:00
Merge pull request #2040 from jlipps/master
don't ever use sudo to reinstall npm modules
This commit is contained in:
@@ -85,7 +85,6 @@ run_cmd_output() {
|
||||
|
||||
reset_npm() {
|
||||
echo "RESETTING NPM"
|
||||
set +e
|
||||
if $hardcore ; then
|
||||
echo "* Removing NPM modules"
|
||||
run_cmd rm -rf node_modules
|
||||
@@ -100,12 +99,6 @@ reset_npm() {
|
||||
echo "* Installing new or updated NPM modules (including devDeps)"
|
||||
run_cmd npm install .
|
||||
fi
|
||||
install_status=$?
|
||||
set -e
|
||||
if [ $install_status -gt 0 ]; then
|
||||
echo "install failed. Trying again with sudo. Only do this if it's not a network error."
|
||||
run_cmd sudo npm install .
|
||||
fi
|
||||
}
|
||||
|
||||
reset_general() {
|
||||
|
||||
Reference in New Issue
Block a user