Thread (61 messages) flat view 61 messages, 2 authors, 2016-06-15

Re: [PATCH v18 16/48] refs.c: add an err argument to delete_ref_loose

From: Ronnie Sahlberg <hidden>
Date: 2016-06-15 23:01:42

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 [off-list ref] wrote:
On 06/19/2014 12:27 AM, Ronnie Sahlberg wrote:
quoted
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.
quoted
I will respin a v19 with these patches reordered.
Michael

--
Michael Haggerty
mhagger@alum.mit.edu
http://softwareswirl.blogspot.com/
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help