comparison hgext/lfs/__init__.py @ 44072:1a6dd50cd0db

lfs: don't skip locally available blobs when verifying The `skipflags` config was introduced in a2ab9ebcd85b, which specifically calls out downloading and storing all blobs as potentially too expensive. But I don't see any reason to skip blobs that are already available locally. Hashing the blob is the only way to indirectly verify the rawdata content stored in the revlog. (The note in that commit about skipping renamed is still correct, but the reason given about needing fulltext isn't.) Differential Revision: https://phab.mercurial-scm.org/D7712
author Matt Harbison <matt_harbison@yahoo.com>
date Sun, 22 Dec 2019 23:50:19 -0500
parents 34e8305f02bd
children 87167caa2989
comparison
equal deleted inserted replaced
44071:34e8305f02bd 44072:1a6dd50cd0db
403 pointers = wrapper.extractpointers(repo, scmutil.revrange(repo, revs)) 403 pointers = wrapper.extractpointers(repo, scmutil.revrange(repo, revs))
404 wrapper.uploadblobs(repo, pointers) 404 wrapper.uploadblobs(repo, pointers)
405 405
406 406
407 @eh.wrapcommand( 407 @eh.wrapcommand(
408 b'verify', opts=[(b'', b'no-lfs', None, _(b'skip all lfs blob content'))] 408 b'verify',
409 opts=[(b'', b'no-lfs', None, _(b'skip missing lfs blob content'))],
409 ) 410 )
410 def verify(orig, ui, repo, **opts): 411 def verify(orig, ui, repo, **opts):
411 skipflags = repo.ui.configint(b'verify', b'skipflags') 412 skipflags = repo.ui.configint(b'verify', b'skipflags')
412 no_lfs = opts.pop('no_lfs') 413 no_lfs = opts.pop('no_lfs')
413 414