# HG changeset patch # User Jun Wu # Date 1513636620 28800 # Node ID 488634db5928282dd0ae665f26a46d1eda049dd9 # Parent d624c8558c6156d92f046b0151973d2ca74262ba lfs: fix committing deleted files caused by e0a1b9ee93cd e0a1b9ee93cd (lfs: add a repo requirement for this extension once an lfs file is committed) introduced a regression that prevents committing file deletion. This patch fixes that. Differential Revision: https://phab.mercurial-scm.org/D1717 diff -r d624c8558c61 -r 488634db5928 hgext/lfs/__init__.py --- a/hgext/lfs/__init__.py Mon Dec 18 13:15:10 2017 -0500 +++ b/hgext/lfs/__init__.py Mon Dec 18 14:37:00 2017 -0800 @@ -124,7 +124,7 @@ if 'lfs' not in repo.requirements: ctx = repo[kwargs['node']] # TODO: is there a way to just walk the files in the commit? - if any(ctx[f].islfs() for f in ctx.files()): + if any(ctx[f].islfs() for f in ctx.files() if f in ctx): repo.requirements.add('lfs') repo._writerequirements() diff -r d624c8558c61 -r 488634db5928 tests/test-lfs.t --- a/tests/test-lfs.t Mon Dec 18 13:15:10 2017 -0500 +++ b/tests/test-lfs.t Mon Dec 18 14:37:00 2017 -0800 @@ -680,3 +680,12 @@ $ hg -R convert_lfs2 config --debug extensions | grep lfs $TESTTMP/convert_lfs2/.hg/hgrc:*: extensions.lfs= (glob) + +Committing deleted files works: + + $ hg init $TESTTMP/repo-del + $ cd $TESTTMP/repo-del + $ echo 1 > A + $ hg commit -m 'add A' -A A + $ hg rm A + $ hg commit -m 'rm A'