Re: [PATCH v2] add test to demonstrate that shallow recursive clones fail
- From
Jens Lehmann <jens.lehmann@web.de>
- Date
- Nov 17, 2015, 21:00 UTC
- Message-ID
- <564B9562.3050801@web.de>
- In-Reply-To
- <CAGZ79kaiWSyXUG_mgUcqt6Mpuj_1EhNOTyG-7NL=28vvi770jA@mail.gmail.com>
Am 17.11.2015 um 21:49 schrieb Stefan Beller:
> I assumed we'd have yet another flag to activate the new behavior, > but if you want to roll out that new feature as a default, I agree on > needing the fallback.
Ah, I was under the impression that users are surprised by --depth not propagating into the submodules, so I considered this a bugfix which doesn't need extra configuration (and the subject line seems to support my impression ;-).
And if some users really need the old behavior, they can remove the --recurse-submodules from the clone and issue a submodule update afterwards (Unless they speak up now and tell us why having --depth only apply to the superproject is a feature ... ;-).