Merge pull request #51 from browserstack/alpine-fix

fix: runShellCommand
browserstack · Jan 6, 2025 · 0d13cfa · 0d13cfa
2 parents 5bfd594 + 06e496a
commit 0d13cfa
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/BrowserStackLocal/BrowserStackLocal/BrowserStackTunnel.cs b/BrowserStackLocal/BrowserStackLocal/BrowserStackTunnel.cs
@@ -57,7 +57,7 @@ static bool IsAlpine()
     {
       try
       {
-        string[] output = Util.RunShellCommand("grep -w 'NAME' /etc/os-release");
+        string[] output = Util.RunShellCommand("grep", "-w \'NAME\' /etc/os-release");
         return output[0]?.ToLower()?.Contains("alpine") ?? false;
       }
       catch (System.Exception ex)

diff --git a/BrowserStackLocal/BrowserStackLocal/Util.cs b/BrowserStackLocal/BrowserStackLocal/Util.cs
@@ -6,13 +6,15 @@ namespace BrowserStack
     public class Util {
 
         // Only Unix Support
-        public static string[] RunShellCommand(string command)
+        public static string[] RunShellCommand(string command, string args = "")
         {
-            ProcessStartInfo psi = new ProcessStartInfo("bash", $"-c \"{command}\"") {
+            ProcessStartInfo psi = new ProcessStartInfo {
                 RedirectStandardOutput = true,
                 RedirectStandardError = true,
                 UseShellExecute = false,
-                CreateNoWindow = true
+                CreateNoWindow = true,
+                FileName = command,
+                Arguments = args
             };
 
             Process process = new Process { StartInfo = psi };