Re: [PATCH 0/2] Avoid rewriting "packed-refs" unnecessarily

2017-10-27 Thread Michael Haggerty
On 10/25/2017 10:10 PM, Eric Sunshine wrote: > On Wed, Oct 25, 2017 at 5:53 AM, Michael Haggerty > wrote: >> Since commit dc39e09942 (files_ref_store: use a transaction to update >> packed refs, 2017-09-08), we've been rewriting the `packed-refs` file >> unnecessarily when

Re: [PATCH 0/2] Avoid rewriting "packed-refs" unnecessarily

2017-10-25 Thread Eric Sunshine
On Wed, Oct 25, 2017 at 5:53 AM, Michael Haggerty wrote: > Since commit dc39e09942 (files_ref_store: use a transaction to update > packed refs, 2017-09-08), we've been rewriting the `packed-refs` file > unnecessarily when deleting a loose reference that has no packed >

[PATCH 0/2] Avoid rewriting "packed-refs" unnecessarily

2017-10-25 Thread Michael Haggerty
Since commit dc39e09942 (files_ref_store: use a transaction to update packed refs, 2017-09-08), we've been rewriting the `packed-refs` file unnecessarily when deleting a loose reference that has no packed counterpart. Add some tests demonstrating this problem, then fix it by teaching