Glen Choo <chooglen@xxxxxxxxxx> writes: > @@ -901,6 +902,9 @@ static void collect_changed_submodules(struct repository *r, > > save_warning = warn_on_object_refname_ambiguity; > warn_on_object_refname_ambiguity = 0; > + /* make sure shallows are read */ > + is_repository_shallow(the_repository); This is presumably to initialize the data structures that some later code reads without first calling is_repository_shallow(). Do we know which part of the later code this is? If yes, it would be better to fix it there.