Merge pull request #125 from NativeScript/yosifov/fix-win-recognize

yyosifov · web-flow · commit da67d996dbc0 · 2017-04-06T14:24:28.000+03:00
Fix check for os win - now returns true for mac - 'darwin'
diff --git a/src/project/nativeScriptCli.ts b/src/project/nativeScriptCli.ts
@@ -57,7 +57,7 @@ export class NativeScriptCli {
 
         // always default to cmd on Windows
         // workaround for issue #121 https://github.com/NativeScript/nativescript-vscode-extension/issues/121
-        if (os.platform().indexOf("win") != -1) {
+        if (utils.getPlatform() === utils.Platform.Windows) {
             this._shellPath = "cmd.exe";
         }
 

Original file line number	Diff line number	Diff line change
`@@ -57,7 +57,7 @@ export class NativeScriptCli {`
`57`	`57`
`58`	`58`	`// always default to cmd on Windows`
`59`	`59`	`// workaround for issue #121 https://github.com/NativeScript/nativescript-vscode-extension/issues/121`
`60`		`- if (os.platform().indexOf("win") != -1) {`
	`60`	`+ if (utils.getPlatform() === utils.Platform.Windows) {`
`61`	`61`	`this._shellPath = "cmd.exe";`
`62`	`62`	`}`
`63`	`63`