Re: [PATCH v13 4/4] ls-remote: create '--sort' option

From: Harald Nordgren <haraldnordgren@gmail.com>
Date: Apr 9, 2018, 01:45 UTC
Message-ID: <CAHwyqnU-hyRboVDG8BBRJU_uURRdoh4UPg7=PjsJ+V9YPwqYcw@mail.gmail.com>
In-Reply-To: <xmqqin916vu6.fsf@gitster-ct.c.googlers.com>

On Mon, Apr 9, 2018 at 2:56 AM, Junio C Hamano <gitster@pobox.com> wrote:

Show 10 quoted lines

>
> With the update since v12, I think "because `ls-remote` deals only
> with remotes," can be dropped entirely, and still convey what needs
> to be told: "Be aware some keys that needs access to objects that
> are not here won't work".
>
> Instead, it is probably a better idea to spend that half-line worth
> of characters to describe in what way they do not work (do they try
> to deref a NULL pointer and dump core?  do they notice we need
> missing objects and give an error?).

Updated the docs again. The specific error we get on commiterdate is

    fatal: missing object f0d0fd3a5985d5e588da1e1d11c85fba0ae132f8 for
refs/pull/10/head

So no core dump. But a fatal error with a possibly confusing message.