Removed call to Setpgid in the process gear.

process/process.go

@@ -52,9 +52,6 @@ func (p Process) Create() *Process {
 	p.command = exec.Command(absolutePath)
 	p.command.Dir = scriptDirectory
 
-	// Do cross-platform things to prepare this process to run
-	PrepareCommandProcess(&p)
-
 	// Copy the current processes ENV and merge in the new ones. We do this
 	// so the sub process gets PATH and stuff. We merge our path in over
 	// the top of the current one so the ENV from Buildkite and the agent

process/utils.go

@@ -5,17 +5,10 @@ package process
 import (
 	"os"
 	"os/exec"
-	"syscall"
 
 	"github.com/kr/pty"
 )
 
 func StartPTY(c *exec.Cmd) (*os.File, error) {
 	return pty.Start(c)
 }
-
-func PrepareCommandProcess(p *Process) {
-	// Children of the forked process will inherit its process group
-	// This is to make sure that all grandchildren dies when this Process instance is killed
-	p.command.SysProcAttr = &syscall.SysProcAttr{Setpgid: true}
-}

process/utils_windows.go

@@ -9,7 +9,3 @@ import (
 func StartPTY(c *exec.Cmd) (*os.File, error) {
 	return nil, errors.New("PTY is not supported on Windows")
 }
-
-func PrepareCommandProcess(p *Process) {
-	// Nothing to prepare!
-}