Re: [PATCH/WIP 03/11] t5403: avoid doing "git add foo/bar" where foo/.git exists

From: Junio C Hamano <gitster@pobox.com>
Date: Oct 27, 2011, 17:41 UTC
Message-ID: <7vobx2z60w.fsf@alter.siamese.dyndns.org>
In-Reply-To: <CACsJy8C2nUJkN5=E7p2u_wjHqWy7EC_BS3Sr4+_QgunWHDdtKg@mail.gmail.com>

Nguyen Thai Ngoc Duy <pclouds@gmail.com> writes:

Show 5 quoted lines

> On Thu, Oct 27, 2011 at 4:26 AM, Junio C Hamano <gitster@pobox.com> 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.