co-exec

co-exec copied to clipboard

It would be great to have access to stderr as well as stdout. Off the cuff, https://github.com/visionmedia/co-exec/blob/master/index.js#L15 could be changed to return an object containing both stdout and stderr.

+1

SGTM

@visionmedia not familiar with that acronym.

sounds good to me :D. If you're interested in sending a pull request with test tweaks that would be awesome

I looked into it, but ended up thinking it might not be a good idea, since I figured that stderr and err are often linked, and then you can't really try/catch nicely anymore? But I may have overlooked something really obvious haha

On Wed, Oct 8, 2014 at 12:10 AM, TJ Holowaychuk [email protected] wrote:

sounds good to me :D. If you're interested in sending a pull request with test tweaks that would be awesome

— Reply to this email directly or view it on GitHub https://github.com/visionmedia/co-exec/issues/6#issuecomment-58305922.

I thought node used stderr or the error message, although you can have stderr for a program that exits successfully as well so it does make sense to separate them. In the case of this sort of utility that's probably not too helpful though, especially since it's a few lines :D haha