Re: [PATCH 1/4 v4] submodules: make submodule-prefix option
- From
- Brandon Williams <bmwill@google.com>
- Date
- Sep 27, 2016, 20:43 UTC
- Message-ID
- <20160927204320.GD32565@google.com>
- In-Reply-To
- <xmqqoa39m67l.fsf@gitster.mtv.corp.google.com>
On 09/27, Junio C Hamano wrote:
Show 17 quoted lines
> Brandon Williams <bmwill@google.com> writes: > > >> s/submodules/submodule-prefix/ at least. > > > > So should the #define be SUPPORT_SUBMODULE_PREFIX instead? That may be > > too narrow minded and not looking toward future submodule options > > support but I'm not sure. > > I am not convinced that this prefix needs to be tied/limited to > submodule, at least not yet, though. I view it as a prefix that > points from above the repository's top, of which submodule support > may be a good sample user, but the caller may not necessarily be > doing or interested in "submodule support". > > That's also part of figuring out how we want define the semantics of > this thing and how we want to present it to the end-users, I guess, > so we may have to rename it when we know more, but that's OK.
K we can leave the name as is for now then. Thankfully it should be a simple name change since it only lives in 1 file.
-- Brandon Williams