Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

os: document that (*Process).Wait only works for subprocesses #7072

Closed
adonovan opened this issue Jan 6, 2014 · 3 comments
Closed

os: document that (*Process).Wait only works for subprocesses #7072

adonovan opened this issue Jan 6, 2014 · 3 comments
Labels
Documentation Issues describing a change to documentation. FrozenDueToAge
Milestone
Go1.3

Comments

@adonovan
Copy link
Member

adonovan commented Jan 6, 2014 

The documentation for:

p, err := os.FindProcess(pid)
p.Wait()

doesn't restrict which processes this works for, but the Linux implementation is based
on wait4(2) which only works for child processes.  It returns an error for unrelated or
parent processes.  This should be documented.
@rsc
Copy link
Contributor

rsc commented Mar 3, 2014

Comment 1:

Labels changed: added release-go1.3, documentation.

Status changed to Accepted.

@bradfitz
Copy link
Contributor

bradfitz commented Mar 6, 2014

Comment 2:

Sent https://golang.org/cl/71900045

Status changed to Started.

@bradfitz
Copy link
Contributor

bradfitz commented Mar 7, 2014

Comment 3:

This issue was closed by revision a35b9e4.

Status changed to Fixed.

@adonovan adonovan added fixed Documentation Issues describing a change to documentation. labels Mar 7, 2014
@rsc rsc added this to the Go1.3 milestone Apr 14, 2015
@rsc rsc removed the release-go1.3 label Apr 14, 2015
@golang golang locked and limited conversation to collaborators Jun 25, 2016
This issue was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
Documentation Issues describing a change to documentation. FrozenDueToAge
Projects
None yet
Milestone
Go1.3
Development

No branches or pull requests
4 participants
@bradfitz @rsc @adonovan @gopherbot