On 04/26, Ævar Arnfjörð Bjarmason wrote:
> This is an expansion of the previously solo 02/05 "clone: add a
> --no-tags option to clone without tags" patch (see
> <20170418191553.15464-1-avarab@xxxxxxxxx>).
>
> This addresses the comments by Junio & Jonathan Nieder on v2 (thanks a
> lot), and in addition implements a --no-tags-submodules option. That
> code was implemented by Brandon & sent to me privately after I'd
> failed to come up with it, but I added tests, a commit message & bash
> completion to it.
Na you would have come up with it, I've just lived in submodule land a
little too long (though not as long as Stephan has!) :D
> The WIP 5/5 patch implements a submodule.NAME.tags config facility for
> the option, but is broken currently & floats along in this submission
> as an RFC patch. AFAICT it *should* work and it goes through all the
> motions the similar existing *.shallow config does, but for some
> reason the tags=false option isn't picked up & propagated in a freshly
> cloned submodule.
>
> I'm probably missing something trivial, but I can't see what it is,
> I'm hoping thath either Stefan or Brandon will see what that is.
Overall the series looks good. I've mentioned in the other threads that
it probably makes more sense to have --recurse-submodules simply pass
through known good commands to its children (e.g. --no-tags) simply
because it makes the UX a little bit easier to work with (I don't have
to remember all the fancy --OPT-submodules stuff, only
--recurse-submodules). That is unless you have some good rational that
I'm not considering (completely possible :D).
>
> Brandon Williams (1):
> clone: add a --no-tags-submodules to pass --no-tags to submodules
>
> Ævar Arnfjörð Bjarmason (4):
> tests: change "cd ... && git fetch" to "cd &&\n\tgit fetch"
> clone: add a --no-tags option to clone without tags
> tests: rename a test having to do with shallow submodules
> WIP clone: add a --[no-]recommend-tags & submodule.NAME.tags config
>
> Documentation/git-clone.txt | 21 ++++
> Documentation/git-submodule.txt | 8 +-
> builtin/clone.c | 19 +++-
> builtin/submodule--helper.c | 21 +++-
> contrib/completion/git-completion.bash | 3 +
> git-submodule.sh | 13 ++-
> submodule-config.c | 8 ++
> submodule-config.h | 1 +
> t/t5612-clone-refspec.sh | 103 +++++++++++++++++---
> ...odules.sh => t5614-clone-submodules-shallow.sh} | 0
> t/t5616-clone-submodules-tags.sh | 106 +++++++++++++++++++++
> 11 files changed, 284 insertions(+), 19 deletions(-)
> rename t/{t5614-clone-submodules.sh => t5614-clone-submodules-shallow.sh} (100%)
> create mode 100755 t/t5616-clone-submodules-tags.sh
>
> --
> 2.11.0
>
--
Brandon Williams