Re: [PATCH 3/5] pack-objects: add --sparse option
- From
Derrick Stolee <stolee@gmail.com>
- Date
- Nov 30, 2018, 15:53 UTC
- Message-ID
- <f9421d46-9cdd-5399-d55e-d5351f149630@gmail.com>
- In-Reply-To
- <xmqq36rjmgqf.fsf@gitster-ct.c.googlers.com>
On 11/29/2018 9:39 PM, Junio C Hamano wrote:
Show 9 quoted lines
> Derrick Stolee <stolee@gmail.com> writes: > >> While _eventually_ we should make this opt-out, we shouldn't do that >> until it has cooked a while. > I actually do not agree. If the knob gives enough benefit, the > users will learn about it viva voce, and in a few more releases > we'll hear "enough users complain that they have to turn it on, > let's make it the default". If that does not happen, the knob > does not deserve to be turned on in the first place.
That's a good philosophy. I'll keep it in mind.
> Having said that, I won't be commenting on this shiny new toy before > the final. I want to see more people help tying the loose ends and > give it final varnish to the upcoming release, as it seems to have > become rockier and larger release than we originally anticipated.
Yeah, no rush on this one. I just wanted to get some initial feedback about the idea.
Thanks, -Stolee