fix 'which' command to detect location of dotnet cli (#622)

GlitchEnzo · Feb 10, 2024 · 5c83956 · 5c83956
1 parent 0d302f6
commit 5c83956
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/src/NuGetForUnity/Editor/Helper/NugetCliHelper.cs b/src/NuGetForUnity/Editor/Helper/NugetCliHelper.cs
@@ -184,7 +184,7 @@ private static string GetDotnetExecutablePath(Encoding outputEncoding)
                     using (var process = Process.Start(
                                              new ProcessStartInfo("which", commandLineToolName)
                                              {
-                                                 RedirectStandardOutput = true, StandardOutputEncoding = outputEncoding,
+                                                 RedirectStandardOutput = true, StandardOutputEncoding = outputEncoding, UseShellExecute = false,
                                              }) ??
                                          throw new InvalidOperationException("Failed to start process."))
                     {

diff --git a/src/NuGetForUnity/Editor/Ui/NugetWindow.cs b/src/NuGetForUnity/Editor/Ui/NugetWindow.cs
@@ -22,6 +22,8 @@ namespace NugetForUnity.Ui
     /// </summary>
     public class NugetWindow : EditorWindow, ISerializationCallbackReceiver
     {
+        private readonly StringBuilder cachedStringBuilder = new StringBuilder();
+
         private readonly Dictionary<string, bool> foldouts = new Dictionary<string, bool>();
 
         /// <summary>
@@ -73,8 +75,6 @@ public class NugetWindow : EditorWindow, ISerializationCallbackReceiver
         /// </summary>
         private List<INugetPackage> availablePackages = new List<INugetPackage>();
 
-        private readonly StringBuilder cachedStringBuilder = new StringBuilder();
-
         /// <summary>
         ///     The currently selected tab in the window.
         /// </summary>