Skip to content

Commit 4baf03e

Browse files
authored
Quick fix for vs dir discovery (#19086)
1 parent ccc7772 commit 4baf03e

File tree

1 file changed

+11
-5
lines changed

1 file changed

+11
-5
lines changed

vsintegration/tests/Salsa/VsMocks.fs

Lines changed: 11 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1653,12 +1653,18 @@ module internal VsActual =
16531653
let vsInstallDir =
16541654
// use the environment variable to find the VS installdir
16551655
let vsvar =
1656-
let var = Environment.GetEnvironmentVariable("VS170COMNTOOLS")
1657-
if String.IsNullOrEmpty var then
1658-
Environment.GetEnvironmentVariable("VSAPPIDDIR")
1656+
// Try VS180COMNTOOLS first, then VS170COMNTOOLS, then VSAPPIDDIR
1657+
// TODO : use tryGetVSInstallDir from test utils instead
1658+
let var18 = Environment.GetEnvironmentVariable("VS180COMNTOOLS")
1659+
if String.IsNullOrEmpty var18 then
1660+
let var17 = Environment.GetEnvironmentVariable("VS170COMNTOOLS")
1661+
if String.IsNullOrEmpty var17 then
1662+
Environment.GetEnvironmentVariable("VSAPPIDDIR")
1663+
else
1664+
var17
16591665
else
1660-
var
1661-
if String.IsNullOrEmpty vsvar then failwith "VS170COMNTOOLS and VSAPPIDDIR environment variables not found."
1666+
var18
1667+
if String.IsNullOrEmpty vsvar then failwith "VS180COMNTOOLS, VS170COMNTOOLS and VSAPPIDDIR environment variables not found."
16621668
Path.Combine(vsvar, "..")
16631669

16641670
let CreateEditorCatalog() =

0 commit comments

Comments
 (0)