On Tue, Aug 24, 2021 at 03:06:55PM -0700, Junio C Hamano wrote: > Junio C Hamano <gitster@xxxxxxxxx> writes: > > > FWIW, here is what I have somewhere in 'seen' where two topics meet. > > Oops, one change missed. Thanks; that matches my own resolution. I noticed that it does fail the new test in t5319, since writing a MIDX wants to make sure that we are only touching an alternate's object directory (which will fail if we are running `git multi-pack-index` from outside of a repository). My opinion is that we should require being inside of a repository to run the MIDX builtin. Otherwise we're allowing that command to modify any old MIDX, which doesn't make sense. I think we probably need a single unifying topic, so I'm happy if you want to discard one of our two topics from seen in the meantime. Thanks, Taylor
