Merge pull request #890 from microsoft/dev/andris/cmake_tools/FixGitH…

…ub886_IgnoreVSKitsOnLinux

Fix for issue 886: ignore vs kits processing when not on windows

src/kit.ts

@@ -725,18 +725,23 @@ export async function scanForClangCLKits(searchPaths: string[]): Promise<Promise
 }
 
 async function getVSInstallForKit(kit: Kit): Promise<VSInstallation|undefined> {
- console.assert(kit.visualStudio);
- console.assert(kit.visualStudioArchitecture);
- const installs = await vsInstallations();
- const match = (inst: VSInstallation) =>
- // old Kit format
- (legacyKitVSName(inst) == kit.visualStudio) ||
- // new Kit format
- (kitVSName(inst) === kit.visualStudio) ||
- // Clang for VS kit format
- (!!kit.compilers && kit.name.indexOf("Clang") >= 0 && kit.name.indexOf(vsDisplayName(inst)) >= 0);
-
- return installs.find(inst => match(inst));
+ if (process.platform !== "win32") {
+ return undefined;
+ }
+
+ console.assert(kit.visualStudio);
+ console.assert(kit.visualStudioArchitecture);
+
+ const installs = await vsInstallations();
+ const match = (inst: VSInstallation) =>
+ // old Kit format
+ (legacyKitVSName(inst) == kit.visualStudio) ||
+ // new Kit format
+ (kitVSName(inst) === kit.visualStudio) ||
+ // Clang for VS kit format
+ (!!kit.compilers && kit.name.indexOf("Clang") >= 0 && kit.name.indexOf(vsDisplayName(inst)) >= 0);
+
+ return installs.find(inst => match(inst));
 }
 
 export async function getVSKitEnvironment(kit: Kit): Promise<Map<string, string>|null> {