Fix: Retrieving dotnet runtimes hangs when there are too many runtime…

…s installed (#119)
ionide · Oct 23, 2021 · ce9f379 · ce9f379
1 parent 1c9317b
commit ce9f379
Showing 1 changed file with 10 additions and 5 deletions.
diff --git a/src/Ionide.ProjInfo/Library.fs b/src/Ionide.ProjInfo/Library.fs
@@ -31,13 +31,18 @@ module SdkDiscovery =
             info.ArgumentList.Add arg
 
         info.RedirectStandardOutput <- true
-        let p = System.Diagnostics.Process.Start(info)
+        use p = System.Diagnostics.Process.Start(info)
+
+        let output =
+            seq {
+                while not p.StandardOutput.EndOfStream do
+                    yield p.StandardOutput.ReadLine()
+            }
+            |> Seq.toArray
+
         p.WaitForExit()
+        output
 
-        seq {
-            while not p.StandardOutput.EndOfStream do
-                yield p.StandardOutput.ReadLine()
-        }
 
     let private (|SemVer|_|) version =
         match SemanticVersioning.Version.TryParse version with