Re: [PATCH] status: read submodule process output before calling wait()

2015-03-22 Thread Jeff King
On Sun, Mar 22, 2015 at 03:44:55AM -0400, Jeff King wrote: > The status code tries to read the output of "git submodule > summary" over a pipe by waiting for the program to finish > and then reading its output, like this: > > run_command(&sm_summary); > len = strbuf_read(&cmd_stdout, sm_summa

[PATCH] status: read submodule process output before calling wait()

2015-03-22 Thread Jeff King
On Sat, Mar 21, 2015 at 10:56:54PM -0700, Wincent Colaiuta wrote: > I've never seen this hang before despite frequent use of submodules. > Oddly, I was able to work around the hang by moving the submodule in > two hops (one from Ansible v1.6.10 to v1.7.0, then from v1.7.0 to > v1.8.4). I am not su