Re: [PATCH v8 38/44] refs.c: pack all refs before we start to rename a ref

From: Ronnie Sahlberg <sahlberg@google.com>
Date: May 22, 2014, 18:02 UTC
Message-ID: <CAL=YDWn8qh7SMHR+T1rsxB8u=Qd6MOSTc9Wo8pUc-jy5tNDdjg@mail.gmail.com>
In-Reply-To: <20140522175153.GQ12314@google.com>

On Thu, May 22, 2014 at 10:51 AM, Jonathan Nieder <jrnieder@gmail.com> wrote:

Show 16 quoted lines

> Ronnie Sahlberg wrote:
>> On Wed, May 21, 2014 at 4:57 PM, Jonathan Nieder <jrnieder@gmail.com> wrote:
>>> Ronnie Sahlberg wrote:
>
>>>> This means that most loose refs will no longer be present after the rename
>>>
>>> Is this to handle the "git branch -m foo/bar foo" case or for some other
>>> purpose?
>>
>> Yes. That is the main reason.
>
> Thanks.  Please say so in the log message.  Remember that people of
> the future debugging and trying to figure out how to fix the latest
> regression without breaking something else and whether to just revert
> your patch cannot read your mind. ;-)
>

Done.