Merge pull request #30095 from Microsoft/nodeExecFileSync
Do not wrap npm path with quotes
This commit is contained in:
commit
4db4c58924
|
@ -89,10 +89,6 @@ namespace ts.server.typingsInstaller {
|
||||||
log);
|
log);
|
||||||
this.npmPath = npmLocation !== undefined ? npmLocation : getDefaultNPMLocation(process.argv[0]);
|
this.npmPath = npmLocation !== undefined ? npmLocation : getDefaultNPMLocation(process.argv[0]);
|
||||||
|
|
||||||
// If the NPM path contains spaces and isn't wrapped in quotes, do so.
|
|
||||||
if (stringContains(this.npmPath, " ") && this.npmPath[0] !== `"`) {
|
|
||||||
this.npmPath = `"${this.npmPath}"`;
|
|
||||||
}
|
|
||||||
if (this.log.isEnabled()) {
|
if (this.log.isEnabled()) {
|
||||||
this.log.writeLine(`Process id: ${process.pid}`);
|
this.log.writeLine(`Process id: ${process.pid}`);
|
||||||
this.log.writeLine(`NPM location: ${this.npmPath} (explicit '${Arguments.NpmLocation}' ${npmLocation === undefined ? "not " : ""} provided)`);
|
this.log.writeLine(`NPM location: ${this.npmPath} (explicit '${Arguments.NpmLocation}' ${npmLocation === undefined ? "not " : ""} provided)`);
|
||||||
|
|
Loading…
Reference in a new issue