]> Cypherpunks repositories - gostls13.git/commit
undo CL 131750044 / 2d6d44ceb80e
authorRuss Cox <rsc@golang.org>
Wed, 19 Nov 2014 19:16:12 +0000 (14:16 -0500)
committerIan Lance Taylor <iant@golang.org>
Wed, 19 Nov 2014 19:16:12 +0000 (14:16 -0500)
commit2d53d6b5d5b5f10c072549607a221c13e5cdb7e3
tree6355ce23d8d3d5ef953f17f4c3f376303482c2dc
parentf03f0cba2ba03d5e0ddb88c7b1f98d7bbbabb9ce
undo CL 131750044 / 2d6d44ceb80e

Breaks reading from stdin in parent after exec with SysProcAttr{Setpgid: true}.

package main

import (
        "fmt"
        "os"
        "os/exec"
        "syscall"
)

func main() {
        cmd := exec.Command("true")
        cmd.SysProcAttr = &syscall.SysProcAttr{Setpgid: true}
        cmd.Run()

        fmt.Printf("Hit enter:")
        os.Stdin.Read(make([]byte, 100))
        fmt.Printf("Bye\n")
}

In go1.3, I type enter at the prompt and the program exits.
With the CL being rolled back, the program wedges at the
prompt.

««« original CL description
syscall: SysProcAttr job control changes

Making the child's process group the foreground process group and
placing the child in a specific process group involves co-ordination
between the parent and child that must be done post-fork but pre-exec.

LGTM=iant
R=golang-codereviews, gobot, iant, mikioh.mikioh
CC=golang-codereviews
https://golang.org/cl/131750044

»»»

LGTM=minux, dneil
R=dneil, minux
CC=golang-codereviews, iant, michael.p.macinnis
https://golang.org/cl/174450043
src/syscall/exec_bsd.go
src/syscall/exec_linux.go