Re: [PATCH v20 00/48] Use ref transactions

2014-07-14 Thread Ronnie Sahlberg
On Tue, Jul 8, 2014 at 11:48 AM, Junio C Hamano wrote: > Michael Haggerty writes: > >> Patches 01-19 -- ACK mhagger >> Patches 20-42 -- I sent various comments, small to large, concerning >> these patches >> Patch 43 -- Needs more justification if it is to be acceptable >> Patch 44 -- Depends on

Re: [PATCH v20 00/48] Use ref transactions

2014-07-14 Thread Ronnie Sahlberg
On Tue, Jul 8, 2014 at 9:29 AM, Michael Haggerty wrote: > On 06/20/2014 04:42 PM, Ronnie Sahlberg wrote: >> This patch series can also be found at >> https://github.com/rsahlberg/git/tree/ref-transactions >> >> This patch series is based on current master and expands on the transaction >> API. It

Re: [PATCH v20 00/48] Use ref transactions

2014-07-08 Thread Jeff King
On Tue, Jul 08, 2014 at 11:48:06AM -0700, Junio C Hamano wrote: > I'd say that "if you have foo/bar you cannot have foo" may have > started as an implementation limitation, but the interoperability > requirement with existing versions of Git and with existing > repositories makes it necessary to e

Re: [PATCH v20 00/48] Use ref transactions

2014-07-08 Thread Junio C Hamano
Michael Haggerty writes: > Patches 01-19 -- ACK mhagger > Patches 20-42 -- I sent various comments, small to large, concerning > these patches > Patch 43 -- Needs more justification if it is to be acceptable > Patch 44 -- Depends on 43 > Patches 45-48 -- I didn't quite get to these, but... > > Pe

Re: [PATCH v20 00/48] Use ref transactions

2014-07-08 Thread Michael Haggerty
On 06/20/2014 04:42 PM, Ronnie Sahlberg wrote: > This patch series can also be found at > https://github.com/rsahlberg/git/tree/ref-transactions > > This patch series is based on current master and expands on the transaction > API. It converts all ref updates, inside refs.c as well as external, to

[PATCH v20 00/48] Use ref transactions

2014-06-20 Thread Ronnie Sahlberg
This patch series can also be found at https://github.com/rsahlberg/git/tree/ref-transactions This patch series is based on current master and expands on the transaction API. It converts all ref updates, inside refs.c as well as external, to use the transaction API for updates. This makes most of