Git-command stuck about grunt-shell

[derpoho]

Same here.

Our Command executes git flow feature start "feature_name" and gets stuck although everything is done correctly.

Any fix?

[derpoho]

@Jpunt After a quick test i suggest you to use grunt-exec in the meantime until this is fixed.
It works for us and the commands are the same.

[silverwind]

Also had a command getting stuck: ncu -u which is npm-check-updates.

[silverwind]

Nevermind me, my issue seems with the module itself - raineorshine/npm-check-updates#119

[zenopopovici]

Nope, it's a grunt-shell issue. Also npm install breaks. This could be the reason: raineorshine/npm-check-updates#119

[silverwind]

I think the issue with ncu boils down to process.stdin.isTTY being undefined in processes spawned by exec, making is-stdin think there i something on stdin when there is not. grunt-shell's attempt to pipe stdin (process.stdin.pipe(cp.stdin)) to the child is fruitless because the exec command already has run by the point this pipe is attached.

I think the proper solution here would be to switch to spawn and define stdio stream handling inside the spawn options. Either that, or it's a node bug with isTTY (though undefined vs false doesn't change much for this bug).

[sindresorhus]

Closing as it seems it's a Node.js core problem and nothing we can do here.

[silverwind]

I think this should be fixed in this module or maybe a wrapper by calling stdin.end() on the ChildProcess. Any child process that waits on stdin will hang until stdin is explicitely closed:

https://nodejs.org/api/child_process.html#child_process_child_stdin