Re: [PATCH 5/9] submodule update: expose parallelism to the user

2015-10-28 Thread Stefan Beller
On Tue, Oct 27, 2015 at 1:59 PM, Junio C Hamano wrote: > And when 0 starts to meaning something special, we would need to > describe that here (and/or submodule.jobs entry in config.txt). > As I already said, I do not think "0 means num_cpus" is a useful > default, and I would

Re: [PATCH 5/9] submodule update: expose parallelism to the user

2015-10-28 Thread Junio C Hamano
Stefan Beller writes: > On Tue, Oct 27, 2015 at 1:59 PM, Junio C Hamano wrote: >> And when 0 starts to meaning something special, we would need to >> describe that here (and/or submodule.jobs entry in config.txt). >> As I already said, I do not think "0

[PATCH 5/9] submodule update: expose parallelism to the user

2015-10-27 Thread Stefan Beller
Expose possible parallelism either via the "--jobs" CLI parameter or the "submodule.jobs" setting. By having the variable initialized to -1, we make sure 0 can be passed into the parallel processing machine, which will then pick as many parallel workers as there are CPUs. Signed-off-by: Stefan

Re: [PATCH 5/9] submodule update: expose parallelism to the user

2015-10-27 Thread Junio C Hamano
Stefan Beller writes: > @@ -374,6 +374,10 @@ for linkgit:git-clone[1]'s `--reference` and `--shared` > options carefully. > clone with a history truncated to the specified number of revisions. > See linkgit:git-clone[1] > > +-j:: > +--jobs:: This probably