Re: [PATCH v10 25/44] receive-pack.c: use a reference transaction for updating the refs

2014-05-27 Thread Ronnie Sahlberg
On Fri, May 23, 2014 at 2:02 PM, Michael Haggerty wrote: > On 05/23/2014 06:14 PM, Ronnie Sahlberg wrote: >> On Fri, May 23, 2014 at 6:49 AM, Michael Haggerty >> wrote: >>> [...] >>> When I combine these two lines of thought, it suggests to me that we >>> could do a better job of supporting *bot

Re: [PATCH v10 25/44] receive-pack.c: use a reference transaction for updating the refs

2014-05-23 Thread Michael Haggerty
On 05/23/2014 06:14 PM, Ronnie Sahlberg wrote: > On Fri, May 23, 2014 at 6:49 AM, Michael Haggerty > wrote: >> [...] >> When I combine these two lines of thought, it suggests to me that we >> could do a better job of supporting *both* use cases. What if the >> transaction object contained not an

Re: [PATCH v10 25/44] receive-pack.c: use a reference transaction for updating the refs

2014-05-23 Thread Ronnie Sahlberg
On Fri, May 23, 2014 at 6:49 AM, Michael Haggerty wrote: > On 05/19/2014 09:02 PM, Ronnie Sahlberg wrote: >> On Sat, May 17, 2014 at 8:35 AM, Michael Haggerty >> wrote: >>> On 05/16/2014 07:37 PM, Ronnie Sahlberg wrote: Wrap all the ref updates inside a transaction to make the update atomic

Re: [PATCH v10 25/44] receive-pack.c: use a reference transaction for updating the refs

2014-05-23 Thread Michael Haggerty
On 05/19/2014 09:02 PM, Ronnie Sahlberg wrote: > On Sat, May 17, 2014 at 8:35 AM, Michael Haggerty > wrote: >> On 05/16/2014 07:37 PM, Ronnie Sahlberg wrote: >>> Wrap all the ref updates inside a transaction to make the update atomic. >>> >>> Signed-off-by: Ronnie Sahlberg >>> --- >>> builtin/r

Re: [PATCH v10 25/44] receive-pack.c: use a reference transaction for updating the refs

2014-05-19 Thread Ronnie Sahlberg
On Sat, May 17, 2014 at 8:35 AM, Michael Haggerty wrote: > On 05/16/2014 07:37 PM, Ronnie Sahlberg wrote: >> Wrap all the ref updates inside a transaction to make the update atomic. >> >> Signed-off-by: Ronnie Sahlberg >> --- >> builtin/receive-pack.c | 20 ++-- >> 1 file changed

Re: [PATCH v10 25/44] receive-pack.c: use a reference transaction for updating the refs

2014-05-17 Thread Michael Haggerty
On 05/16/2014 07:37 PM, Ronnie Sahlberg wrote: > Wrap all the ref updates inside a transaction to make the update atomic. > > Signed-off-by: Ronnie Sahlberg > --- > builtin/receive-pack.c | 20 ++-- > 1 file changed, 10 insertions(+), 10 deletions(-) > > diff --git a/builtin/rec

[PATCH v10 25/44] receive-pack.c: use a reference transaction for updating the refs

2014-05-16 Thread Ronnie Sahlberg
Wrap all the ref updates inside a transaction to make the update atomic. Signed-off-by: Ronnie Sahlberg --- builtin/receive-pack.c | 20 ++-- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/builtin/receive-pack.c b/builtin/receive-pack.c index c323081..5534138 100