From: Junio C Hamano <gitster@pobox.com>
Date: Thu, 04 Feb 2016 21:57:06 GMT
Subject: Re: [PATCH 6/8] git submodule update: have a dedicated helper for cloning
Message-ID: <xmqq4mdogmvh.fsf@gitster.mtv.corp.google.com>
In-Reply-To: <CAGZ79kb-G=GHdtcH323LbHQ4V_azif04eHVuf8Z3gCeb-e=iKQ@mail.gmail.com>

Stefan Beller <sbeller@google.com> writes:

> I do not think that is a better place as not every consumer of module_list
> (and module_list_compute as the nested function) will need to use the
> submodule caching API.

Ah, if that is the case, then OK.

I was imagining that module-list may someday would return a list of
"struct" instances each of which describes a submodule and has .name
and .path fields, or something, instead of just being a "here is the
path to the submodule, go look it up yourself" interface.  But we
are not there (at least not yet--and I am not saying we agreed that
we would eventually want to go there)