On Fri, Oct 23, 2015 at 12:25 PM, Junio C Hamano <gitster@xxxxxxxxx> wrote: > Stefan Beller <sbeller@xxxxxxxxxx> writes: > >> submodule update: Expose parallelism to the user >> clone: Allow an explicit argument for parallel submodule clones > > downcase Expose and Allow, perhaps? Will do, thanks! > > > > I was looking at the previous one and I am getting the feeling that > everything up to "run-command: fix missing output from late callbacks" > is ready for 'next'. Am I being too optimistic and missing something > that may make you want to do another reroll? I would even argue for "submodule config: keep update strategy around" to be ready for next. ;) But as that is quite unrelated to the previous patches and only needed for the last patch, we can omit that safely too. All the fixes up to "run-command: fix missing output from late callbacks" sound good to me for next. I have run into a problem cloning big repositories though, but I haven't found the problem. So the whole parallel processing machine may need another bug fix later on. > > 37bc721 submodule.c: write "Fetching submodule <foo>" to stderr > 0904370 xread: poll on non blocking fds > fd6ed7c xread_nonblock: add functionality to read from fds without blocking > e7ba957 strbuf: add strbuf_read_once to read without blocking > 8fc3f2e sigchain: add command to pop all common signals > f57c806 run-command: add an asynchronous parallel child processor > 4733d9e fetch_populated_submodules: use new parallel job processing > dca8113 submodules: allow parallel fetching, add tests and documentation > 79f3857 run-command: fix early shutdown > 1c53754 run-command: clear leftover state from child_process structure > 63ce47e run-command: initialize the shutdown flag > c3a5d11 test-run-command: test for gracefully aborting > 74cc04d test-run-command: increase test coverage > 376d400 run-command: fix missing output from late callbacks -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html