On Mon, Jun 21, 2021 at 06:25:15PM -0400, Taylor Blau wrote: > When writing a new multi-pack index, write_midx_internal() attempts to > clean up any auxiliary files (currently just the MIDX's `.rev` file, but > soon to include a `.bitmap`, too) corresponding to the MIDX it's > replacing. > > This step should happen after the new MIDX is written into place, since > doing so beforehand means that the old MIDX could be read without its > corresponding .rev file. Good catch. The patch looks obviously correct. -Peff
