win: more robust parsing of SDK version

Fixes: nodejs#1179
refack · May 13, 2017 · d067e3a · d067e3a
1 parent ce815f9
commit d067e3a
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/lib/Find-VS2017.cs b/lib/Find-VS2017.cs
@@ -228,9 +228,10 @@ private static bool CheckInstance(ISetupInstance2 setupInstance2, ref StringBuil
                     hasMSBuild = true;
                 else if (id == "Microsoft.VisualStudio.Component.VC.Tools.x86.x64")
                     hasVCTools = true;
-                else if (id.StartsWith(Win10SDKPrefix))
-                    Win10SDKVer = Math.Max(Win10SDKVer, UInt32.Parse(id.Substring(Win10SDKPrefix.Length)));
-                else if (id == "Microsoft.VisualStudio.Component.Windows81SDK")
+                else if (id.StartsWith(Win10SDKPrefix)) {
+                    string[] parts = id.ToString().Substring(Win10SDKPrefix.Length).Split('.');
+                    Win10SDKVer = Math.Max(Win10SDKVer, UInt32.Parse(parts[0]));
+                } else if (id == "Microsoft.VisualStudio.Component.Windows81SDK")
                     hasWin8SDK = true;
                 else
                     continue;