From: Brad Fitzpatrick Date: Thu, 6 Jul 2017 17:56:21 +0000 (+0000) Subject: os/exec: clarify Cmd.Wait documentation a bit more explicitly X-Git-Tag: go1.9rc1~95 X-Git-Url: http://www.git.cypherpunks.su/?a=commitdiff_plain;h=23ae7a70f9f28fe34547d2857e5efa789625052a;p=gostls13.git os/exec: clarify Cmd.Wait documentation a bit more explicitly It already implied that Cmd.Wait is more than os.Process.Wait, but say so explicitly. See https://github.com/golang/go/issues/18874#issuecomment-309921486 Updates #18874 Change-Id: Iaa46defd776ae0be817d9f4466a99ac78cfd672b Reviewed-on: https://go-review.googlesource.com/47650 Reviewed-by: Russ Cox --- diff --git a/src/os/exec/exec.go b/src/os/exec/exec.go index c3599681c3..893d8ee99a 100644 --- a/src/os/exec/exec.go +++ b/src/os/exec/exec.go @@ -416,8 +416,10 @@ func (e *ExitError) Error() string { return e.ProcessState.String() } -// Wait waits for the command to exit. -// It must have been started by Start. +// Wait waits for the command to exit and waits for any copying to +// stdin or copying from stdout or stderr to complete. +// +// The command must have been started by Start. // // The returned error is nil if the command runs, has no problems // copying stdin, stdout, and stderr, and exits with a zero exit