Merge pull request #209 from articulate/fix/remove-set-pgid

fix: remove set group pid
articulate · May 15, 2024 · 4fa79e4 · 4fa79e4
2 parents b9139af + e1b3347
commit 4fa79e4
Showing 1 changed file with 1 addition and 4 deletions.
diff --git a/main.go b/main.go
@@ -151,9 +151,6 @@ func run(ctx context.Context, name string, args, env []string, l *slog.Logger) i
 	cmd.Stdin = os.Stdin
 	cmd.Stderr = os.Stderr
 	cmd.Stdout = os.Stdout
-	if name != "bash" && name != "sh" && name != "zsh" && name != "fish" {
-		cmd.SysProcAttr = &syscall.SysProcAttr{Setpgid: true}
-	}
 
 	if err := cmd.Start(); err != nil {
 		l.ErrorContext(ctx, "Could not start command", "error", err, "cmd", cmd.String())
@@ -182,7 +179,7 @@ func run(ctx context.Context, name string, args, env []string, l *slog.Logger) i
 	go func() {
 		<-exitch
 		time.Sleep(killWait)
-		l.DebugContext(ctx, "Terminating unrespnsive process", "cmd", cmd.String())
+		l.WarnContext(ctx, "Terminating unrespnsive process", "cmd", cmd.String())
 		cancel()
 	}()