D4097: narrow: remove a repo file-cache invalidation
martinvonz (Martin von Zweigbergk)
phabricator at mercurial-scm.org
Fri Aug 3 21:24:25 UTC 2018
martinvonz created this revision.
Herald added a subscriber: mercurial-devel.
Herald added a reviewer: hg-reviewers.
REVISION SUMMARY
It's unclear why this was needed. All tests pass without it. I asked
Kyle Lippincott (who added the check) and he also doesn't remember
what it was for.
REPOSITORY
rHG Mercurial
REVISION DETAIL
https://phab.mercurial-scm.org/D4097
AFFECTED FILES
mercurial/narrowspec.py
CHANGE DETAILS
diff --git a/mercurial/narrowspec.py b/mercurial/narrowspec.py
--- a/mercurial/narrowspec.py
+++ b/mercurial/narrowspec.py
@@ -135,9 +135,6 @@
# Treat "narrowspec does not exist" the same as "narrowspec file exists
# and is empty".
if e.errno == errno.ENOENT:
- # Without this the next call to load will use the cached
- # non-existence of the file, which can cause some odd issues.
- repo.invalidate(clearfilecache=True)
return set(), set()
raise
return _parsestoredpatterns(spec)
To: martinvonz, #hg-reviewers
Cc: mercurial-devel
More information about the Mercurial-devel
mailing list