Skip to content

Fix childprocess - #2825

Closed
skovhus wants to merge 3 commits into
facebook:masterfrom
skovhus:fix-childprocess
Closed

skovhus wants to merge 3 commits into
facebook:masterfrom
skovhus:fix-childprocess

Conversation

@skovhus

@skovhus skovhus commented Nov 16, 2016

Copy link
Copy Markdown

This fixes #2824

I would wait merging it before nodejs/node#9637 has been approved.

@skovhus

skovhus commented Nov 19, 2016

Copy link
Copy Markdown
Author

The pull request on node has been merged. So this pull should be ready now.

@skovhus

skovhus commented Nov 19, 2016

Copy link
Copy Markdown
Author

Not sure if this pull request should be merged first: #2567

@skovhus

skovhus commented Nov 27, 2016 •

Copy link
Copy Markdown
Author

@mroch Anything I can do to help?

@gabelevi

gabelevi commented Dec 2, 2016

Copy link
Copy Markdown
Contributor

Sorry for the delay! I've rebased this on the other work and made sure that execSync, execFileSync and spawnSync all can now take a string as well. I'll get this merged ASAP!

@facebook-github-bot

Copy link
Copy Markdown
Contributor

@gabelevi has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

child_process

4 participants