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

Let me redo it so it applies to current master. On Mon, Jun 16, 2014 at 5:22 PM, Junio C Hamano gits...@pobox.com wrote: The series applies cleanly up to 25/48 or so to 'master', but this step already breaks tests, at least t1400 but possibly others. Please do not make me bisect X-. Thanks.

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

Thanks. I have rebased it on current master and re-sent it as v18 I passes all tests, except t0008 which is unrelated to this series. regards ronnie sahlberg On Tue, Jun 17, 2014 at 7:34 AM, Ronnie Sahlberg sahlb...@google.com wrote: Let me redo it so it applies to current master. On Mon,

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

Ronnie Sahlberg sahlb...@google.com writes: Thanks. I have rebased it on current master and re-sent it as v18 I passes all tests, except t0008 which is unrelated to this series. I think You need to either have 97c1364b (t0008: do not depend on 'echo' handling backslashes specially,

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

Add an err argument to delete_loose_ref so that we can pass a descriptive error string back to the caller. Pass the err argument from transaction commit to this function so that transaction users will have a nice error string if the transaction failed due to delete_loose_ref. Add a new function

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

The series applies cleanly up to 25/48 or so to 'master', but this step already breaks tests, at least t1400 but possibly others. Please do not make me bisect X-. Thanks. -- To unsubscribe from this list: send the line unsubscribe git in the body of a message to majord...@vger.kernel.org More