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 mhag...@alum.mit.edu 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

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 gits...@pobox.com wrote: Michael Haggerty mhag...@alum.mit.edu 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

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 use

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

2014-07-08 Thread Junio C Hamano
Michael Haggerty mhag...@alum.mit.edu 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

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

[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