Re: [PATCHv7 0/8] Expose submodule parallelism to the user
- From
Junio C Hamano <gitster@pobox.com>
- Date
- Jan 13, 2016, 16:56 UTC
- Message-ID
- <xmqq37u1quyf.fsf@gitster.mtv.corp.google.com>
- In-Reply-To
- <CAGZ79kZO=qLXK=tDt=DT=9og=aNgxk+ukoO4e5qDT=OAJCCS2A@mail.gmail.com>
Stefan Beller <sbeller@google.com> writes:
> Additionally to that I needed to switch the order of arguments for the > parallel processing calls, too, to make it compile again.
That sounds as if the previous one that was queued did not compile, but I didn't recall seeing such breakage. Also comparison between corresponding patches did not show such a change, either.
But perhaps ...
> I assume I just did that work twice as the previous version > ought to compile, too.
... means that you didn't need such a switch after all, and the re-sent series didn't have it.
I am just making sure that I got the right version in my tree, so please don't behave as if I am accusing you that you advertised that you did more work than you actually did and go defensive ;-) If the reason I didn't see one kind of change in the series I received is because it didn't have to be done and the description mistakenly said that such a change is included, that is perfectly fine.
Thanks.