From dad1b5c702a5408d47fdc7d622913be348a22731 Mon Sep 17 00:00:00 2001 From: Russ Cox Date: Wed, 15 Oct 2014 13:09:59 -0400 Subject: os/exec: document that Stdin goroutine must finish in Wait Fixes issue 7990. LGTM=iant, bradfitz R=bradfitz, iant, robryk CC=golang-codereviews https://codereview.appspot.com/156220043 --- src/os/exec/exec.go | 11 +++++++++-- 1 file changed, 9 insertions(+), 2 deletions(-) (limited to 'src/os') diff --git a/src/os/exec/exec.go b/src/os/exec/exec.go index 4aded4171..72b4905d5 100644 --- a/src/os/exec/exec.go +++ b/src/os/exec/exec.go @@ -55,8 +55,15 @@ type Cmd struct { // calling process's current directory. Dir string - // Stdin specifies the process's standard input. If Stdin is - // nil, the process reads from the null device (os.DevNull). + // Stdin specifies the process's standard input. + // If Stdin is nil, the process reads from the null device (os.DevNull). + // If Stdin is an *os.File, the process's standard input is connected + // directly to that file. + // Otherwise, during the execution of the command a separate + // goroutine reads from Stdin and delivers that data to the command + // over a pipe. In this case, Wait does not complete until the goroutine + // stops copying, either because it has reached the end of Stdin + // (EOF or a read error) or because writing to the pipe returned an error. Stdin io.Reader // Stdout and Stderr specify the process's standard output and error. -- cgit v1.2.1