Merge pull request #43 from lafriks-fork/fix/kill_linux

EEParker · web-flow · commit 93b6577537fe · 2020-01-30T07:03:28.000-06:00
Fix killing previous process on Linux and OSX
diff --git a/src/VueCliMiddleware/Util/KillPort.cs b/src/VueCliMiddleware/Util/KillPort.cs
@@ -175,12 +175,14 @@ public static bool Kill(int pid, bool force = false, bool tree = true)
                 if (RuntimeInformation.IsOSPlatform(OSPlatform.OSX))
                 {
                     if (force) { args.Add("-9"); }
-                    RunProcessReturnOutput("kill", "");
+                    args.Add(pid.ToString());
+                    RunProcessReturnOutput("kill", string.Join(" ", args));
                 }
                 else if (RuntimeInformation.IsOSPlatform(OSPlatform.Linux))
                 {
                     if (force) { args.Add("-9"); }
-                    RunProcessReturnOutput("kill", "");
+                    args.Add(pid.ToString());
+                    RunProcessReturnOutput("kill", string.Join(" ", args));
                 }
                 else
                 {

Original file line number	Diff line number	Diff line change
`@@ -175,12 +175,14 @@ public static bool Kill(int pid, bool force = false, bool tree = true)`
`175`	`175`	`if (RuntimeInformation.IsOSPlatform(OSPlatform.OSX))`
`176`	`176`	`{`
`177`	`177`	`if (force) { args.Add("-9"); }`
`178`		`- RunProcessReturnOutput("kill", "");`
	`178`	`+ args.Add(pid.ToString());`
	`179`	`+ RunProcessReturnOutput("kill", string.Join(" ", args));`
`179`	`180`	`}`
`180`	`181`	`else if (RuntimeInformation.IsOSPlatform(OSPlatform.Linux))`
`181`	`182`	`{`
`182`	`183`	`if (force) { args.Add("-9"); }`
`183`		`- RunProcessReturnOutput("kill", "");`
	`184`	`+ args.Add(pid.ToString());`
	`185`	`+ RunProcessReturnOutput("kill", string.Join(" ", args));`
`184`	`186`	`}`
`185`	`187`	`else`
`186`	`188`	`{`