From: Junio C Hamano Date: Thu, 27 Oct 2011 17:41:19 GMT Subject: Re: [PATCH/WIP 03/11] t5403: avoid doing "git add foo/bar" where foo/.git exists Message-ID: <7vobx2z60w.fsf@alter.siamese.dyndns.org> In-Reply-To: Nguyen Thai Ngoc Duy writes: > On Thu, Oct 27, 2011 at 4:26 AM, Junio C Hamano wrote: > ... >> Where did you get this idea that submodule is somehow involved in this test? > > Because "clone2" looks like a submodule (it has ".git" inside with valid HEAD) But there is a crucial difference between "it looks like" and "it is". If it is not a submodule, and the established behaviour is not to treat it like a submodule, then we shouldn't suddenly change our behaviour to start treating as one. > ... Should we stick with one way only? Whatever we have been doing should not change, especially in corner cases.