From: Junio C Hamano <gitster@pobox.com>
Date: Mon, 15 Jun 2015 19:45:30 GMT
Subject: Re: [PATCH v2 06/12] prune_remote(): use delete_refs()
Message-ID: <CAPc5daXJ-FJ_1=EJ8+5t+-fY-w1cQraKYUrvqTu_rXptx24frw@mail.gmail.com>
In-Reply-To: <20150615183906.GA10080@peff.net>

On Mon, Jun 15, 2015 at 11:39 AM, Jeff King <peff@peff.net> wrote:
> On Mon, Jun 15, 2015 at 11:35:28AM -0700, Junio C Hamano wrote:
>
>> But the log message puzzles me.  Didn't refs that cannot be deleted
>> cause the original to fail?  After repacking without these refs, it
>> called delete-ref, and a failure to delete or commit the deletion
>> would have hit the error() down there, no?
>
> I think this discussion:
>
>   http://thread.gmane.org/gmane.comp.version-control.git/271017/focus=271164
>
> from v1 is relevant.

Ah, OK, that makes sense; the log message may want to explain that though.