fix(lib/npm): do not clobber config.execPath

wraithgar · wraithgar · commit 567c9bd03a76 · 2021-02-02T09:11:21.000-08:00
If we were not asked to clobber config.execPath, don't do it. Credit: @wraithgar Reviewed-by: @isaacs
diff --git a/lib/npm.js b/lib/npm.js
@@ -173,8 +173,8 @@ const npm = module.exports = new class extends EventEmitter {
     if (node && node.toUpperCase() !== process.execPath.toUpperCase()) {
       log.verbose('node symlink', node)
       process.execPath = node
+      this.config.execPath = node
     }
-    this.config.execPath = node
 
     await this.config.load()
     this.argv = this.config.parsedArgv.remain

Original file line number	Diff line number	Diff line change
`@@ -173,8 +173,8 @@ const npm = module.exports = new class extends EventEmitter {`
`173`	`173`	`if (node && node.toUpperCase() !== process.execPath.toUpperCase()) {`
`174`	`174`	`log.verbose('node symlink', node)`
`175`	`175`	`process.execPath = node`
	`176`	`+ this.config.execPath = node`
`176`	`177`	`}`
`177`		`- this.config.execPath = node`
`178`	`178`
`179`	`179`	`await this.config.load()`
`180`	`180`	`this.argv = this.config.parsedArgv.remain`