Re: [PATCH v2 00/12] Fix some reference-related races

2013-06-20 Thread Jeff King
On Thu, Jun 20, 2013 at 11:09:49AM +0200, Michael Haggerty wrote: > If the packed-refs file is already locked by another process (and there > is no reason why that cannot be, and there is only one attempt to > acquire the lock), then repack_without_ref() emits an error and returns > with an error

Re: [PATCH v2 00/12] Fix some reference-related races

2013-06-20 Thread Michael Haggerty
On 06/19/2013 08:56 PM, Jeff King wrote: > On Wed, Jun 19, 2013 at 09:51:21AM +0200, Michael Haggerty wrote: > >> Re-roll of mh/ref-races. Thanks to Peff, Junio, and Ramsay for >> reviewing v1. > > Thanks. I just read through them again. Everything looks good to me. > > Patches 10 and 11 are mi

Re: [PATCH v2 00/12] Fix some reference-related races

2013-06-19 Thread Jeff King
On Wed, Jun 19, 2013 at 09:51:21AM +0200, Michael Haggerty wrote: > Re-roll of mh/ref-races. Thanks to Peff, Junio, and Ramsay for > reviewing v1. Thanks. I just read through them again. Everything looks good to me. Patches 10 and 11 are missing my signoff, but obviously: Signed-off-by: Jeff

[PATCH v2 00/12] Fix some reference-related races

2013-06-19 Thread Michael Haggerty
Re-roll of mh/ref-races. Thanks to Peff, Junio, and Ramsay for reviewing v1. Changes since v1: * mh/reflife has graduated to master, so this patch series now applies directly to master. * Some trivial constness adjustments were necessary because of 21a6b9fa read-cache: mark cache_entry p