Skip to content

os/exec: Unclear documentation #6839

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
Closed
bamiaux opened this issue Nov 27, 2013 · 4 comments
Closed

os/exec: Unclear documentation #6839

bamiaux opened this issue Nov 27, 2013 · 4 comments
Labels
FrozenDueToAge Suggested Issues that may be good for new contributors looking for work to do.
Milestone
Unplanned

Comments

@bamiaux
Copy link

bamiaux commented Nov 27, 2013 

os/exec documentation does not mention whether one *must* eventually call Cmd.Wait() on
a *exec.Cmd which has been started with Start().
Looking at the code, it seems to be closing descriptors, so system descriptors may not
be released correctly if we never call Cmd.Wait().
@rsc
Copy link
Contributor

rsc commented Dec 4, 2013

Comment 1:

Labels changed: added release-none, removed go1.3maybe.

@rsc
Copy link
Contributor

rsc commented Dec 4, 2013

Comment 2:

Labels changed: added repo-main.

@davecheney
Copy link
Contributor

Comment 3:

Labels changed: added suggested.

Status changed to Accepted.

@bamiaux bamiaux added accepted Suggested Issues that may be good for new contributors looking for work to do. labels Feb 4, 2014
@rsc rsc added this to the Unplanned milestone Apr 10, 2015
@ALTree
Copy link
Member

ALTree commented Jul 11, 2017

This was fixed by d4b6a19.

@ALTree ALTree closed this as completed Jul 11, 2017
@golang golang locked and limited conversation to collaborators Jul 11, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
FrozenDueToAge Suggested Issues that may be good for new contributors looking for work to do.
Projects
None yet
Milestone
Unplanned
Development

No branches or pull requests
5 participants
@davecheney @rsc @bamiaux @ALTree @gopherbot