search

jakejs / jake

JavaScript build tool, similar to Make or Rake. Built to work with Node.js.
http://jakejs.com
Apache License 2.0
1.97k stars 190 forks source link

Printing stderr when using interactive exec commands by default #182

Closed mblackshaw closed 11 years ago

mblackshaw commented 11 years ago

It took me a while to realize why { interactive: true } wasn't printing out stderr, because I'd assumed the documentation which states that interactive "trumps" printStderr would default it to true.

mde commented 11 years ago

I'm on vacation through Thursday -- I'll take a look at this as soon as I get back.

On Tue, Apr 2, 2013 at 1:59 PM, Matthew Blackshaw notifications@github.comwrote:

It took me a while to realize why { interactive: true } wasn't printing out stderr, because I'd assumed the documentation which states that

interactive "trumps" printStderr would default it to true.

You can merge this Pull Request by running

git pull https://github.com/mblackshaw/jake master

Or view, comment on, or merge it at:

https://github.com/mde/jake/pull/182 Commit Summary

  • Printing out stderr when using interactive exec commands.

File Changes

  • M lib/utils/index.jshttps://github.com/mde/jake/pull/182/files#diff-0(19)

Patch Links:

mde commented 11 years ago

That was a long damn vacation. Sorry it took so long to get to this. And thanks. :)

mde commented 11 years ago

Pushed to NPM in v0.6.8.