From: Junio C Hamano <gitster@pobox.com>
Date: Mon, 20 Sep 2021 18:09:05 GMT
Subject: Re: [PATCH v2 1/1] submodule--helper: fix incorrect newlines in an error message
Message-ID: <xmqqzgs7azlq.fsf@gitster.g>
In-Reply-To: <20210918193116.310575-2-kaartic.sivaraam@gmail.com>

Kaartic Sivaraam <kaartic.sivaraam@gmail.com> writes:

> A refactoring[1] done as part of the recent conversion of
> 'git submodule add' to builtin, changed the error message
> shown when a Git directory already exists locally for a submodule
> name. Before the refactoring, the error used to appear like so:
> ...
> As one could observe the remote information is printed along with the
> first line rather than on its own line. Also, there's an additional
> newline following output.
>
> Make the error message consistent with the error message that used to be
> printed before the refactoring.

Makes sense.  Atharva, an ack?