On Wed, Apr 26, 2017 at 4:12 PM, Ævar Arnfjörð Bjarmason <avarab@xxxxxxxxx> wrote: > @@ -855,7 +856,7 @@ static int prepare_to_clone_next_submodule(const struct cache_entry *ce, > argv_array_pushl(&child->args, "--prefix", suc->prefix, NULL); > if (suc->recommend_shallow && sub->recommend_shallow == 1) > argv_array_push(&child->args, "--depth=1"); > - if (suc->no_tags) > + if (suc->no_tags || suc->recommend_tags == 0) > argv_array_push(&child->args, "--no-tags"); Here you would also need to pay attention to sub->recommend_tags?