Skip to content
This repository has been archived by the owner on Apr 22, 2023. It is now read-only.

spawn() stdio api #1884

Closed
piscisaureus opened this issue Oct 14, 2011 · 2 comments
Closed

spawn() stdio api #1884

piscisaureus opened this issue Oct 14, 2011 · 2 comments

Comments

@piscisaureus
Copy link

  • child_process.spawn() takes stdinStream, stdoutStream, stderrStream as options. This api is not really for public consumption but we should not make it that misleading. In fact it doesn't take a stream, it takes a wrap (only a pipe wrap actually works I think).
  • The customFds api mostly got removed, we only accept -1,0,1,2 as file descriptors now so the child can still access the tty. We should simplify this by letting the user set the inheritStdin etc. option so the child process just inherits whatever stdio the parent is using.
@ghost ghost assigned piscisaureus Oct 14, 2011
@isaacs isaacs closed this as completed Oct 16, 2011
@ry ry reopened this Nov 2, 2011
@piscisaureus
Copy link
Author

@isaacs: I don't know why this was closed. Reopening.

@isaacs
Copy link

isaacs commented Nov 4, 2011

@piscisaureus Yeah, looks like I closed it... I don't remember doing that. I blame gremlins.

@ry ry closed this as completed in 5c3954a Nov 4, 2011
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants