From: Ronnie Sahlberg <sahlberg@google.com>
Date: Thu, 19 Jun 2014 15:56:58 GMT
Subject: Re: [PATCH v18 16/48] refs.c: add an err argument to delete_ref_loose
Message-ID: <CAL=YDW=Xj49fxUvuFU8s-9c6iB=rVoc6+AAbKYuBVMvSPGznOA@mail.gmail.com>
In-Reply-To: <53A21500.2050508@alum.mit.edu>

I have resent v19 that does
1, remove the spurios redundant errno line
2, fixes the type
and
3, reorders the patch as mentioned previously in this thread.

This I hope will be the final version of the series we will need.

regards
ronnie sahlberg

On Wed, Jun 18, 2014 at 3:38 PM, Michael Haggerty <mhagger@alum.mit.edu> wrote:
> On 06/19/2014 12:27 AM, Ronnie Sahlberg wrote:
>> It looks like we need to reorder two of the patches.
>> This patch needs to be moved to later in the series and happen after
>> the delete_ref conversion :
>>
>> refs.c: make delete_ref use a transaction
>> refs.c: add an err argument to delete_ref_loose
>
> That agrees with what I have found out since my first email.  The
> failures go away starting with the later commit that you mentioned.
>
>> I will respin a v19 with these patches reordered.
>
> Michael
>
> --
> Michael Haggerty
> mhagger@alum.mit.edu
> http://softwareswirl.blogspot.com/