On Wed, Jul 26, 2017 at 4:42 PM, brian m. carlson <sand...@crustytoothpaste.net> wrote: > On Wed, Jul 26, 2017 at 04:29:58PM -0700, Jonathan Tan wrote: >> This is an updated version of my previous patch [1], but without the >> list of promises. It is somewhat different now because fsck cannot just >> mark all promised objects as HAS_OBJ. >> >> This could be adapted and incorporated into patch sets that have objects >> missing from the local repo. >> >> I split this up into 4 patches so that you can see how the changes in >> fsck are being tested, but eventually these should probably be combined >> into 1 patch. > > I looked at this and I like the direction it's going. It's pretty > simple and straightforward, which I also like.
I agree.
