tools: fix ESLint update scripts

Changes to the npm dependency in the core repo require changes to update-eslint.sh and update-bable-eslint.sh. PR-URL: #40394 Reviewed-By: Michaël Zasso <[email protected]> Reviewed-By: Antoine du Hamel <[email protected]> Reviewed-By: Luigi Pinca <[email protected]>
nodejs · Oct 12, 2021 · 0a1b399 · 0a1b399
1 parent 40db88b
commit 0a1b399
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/tools/update-babel-eslint.sh b/tools/update-babel-eslint.sh
@@ -5,6 +5,8 @@
 # This script must be be in the tools directory when it runs because it uses
 # $0 to determine directories to work in.
 
+set -e
+
 cd "$( dirname "${0}" )" || exit
 rm -rf node_modules/@babel
 mkdir babel-eslint-tmp
@@ -13,7 +15,7 @@ cd babel-eslint-tmp || exit
 ROOT="$PWD/../.."
 [ -z "$NODE" ] && NODE="$ROOT/out/Release/node"
 [ -x "$NODE" ] || NODE=`command -v node`
-NPM="$ROOT/deps/npm"
+NPM="$ROOT/deps/npm/bin/npm-cli.js"
 
 "$NODE" "$NPM" init --yes
 "$NODE" "$NPM" install --global-style --no-bin-links --production --no-package-lock @babel/core @babel/eslint-parser@latest @babel/plugin-syntax-class-properties@latest @babel/plugin-syntax-import-assertions@latest @babel/plugin-syntax-top-level-await@latest

diff --git a/tools/update-eslint.sh b/tools/update-eslint.sh
@@ -5,6 +5,8 @@
 # This script must be in the tools directory when it runs because it uses the
 # script source file path to determine directories to work in.
 
+set -e
+
 cd "$( dirname "$0" )" || exit
 rm -rf node_modules/eslint node_modules/eslint-plugin-markdown
 (
@@ -14,7 +16,7 @@ rm -rf node_modules/eslint node_modules/eslint-plugin-markdown
     ROOT="$PWD/../.."
     [ -z "$NODE" ] && NODE="$ROOT/out/Release/node"
     [ -x "$NODE" ] || NODE=`command -v node`
-    NPM="$ROOT/deps/npm"
+    NPM="$ROOT/deps/npm/bin/npm-cli.js"
 
     "$NODE" "$NPM" init --yes