diff --git a/mercurial/context.py b/mercurial/context.py --- a/mercurial/context.py +++ b/mercurial/context.py @@ -2214,6 +2214,9 @@ def commit(self, text, parents=None, date=None, extra=None, editor=None, user=None, branch=None): + # We need to discard any keys that are actually clean before the empty + # commit check. + self._compact() allowemptycommit = (len(self._cache) or self._repo.ui.configbool('ui', 'allowemptycommit')) if not allowemptycommit: @@ -2234,6 +2237,29 @@ def clean(self): self._cache = {} + def _compact(self): + """Removes keys from the cache that are actually clean, by comparing + them with the underlting context. + + This can occur during the merge process, e.g. by passing --tool :local + to resolve a conflict. + """ + keys = [] + for path in self._cache.keys(): + cache = self._cache[path] + try: + underlying = self._wrappedctx[path] + if (underlying.data() == cache['data'] and + underlying.flags() == cache['flags']): + keys.append(path) + except error.ManifestLookupError: + # Path not in the underlying manifest (created). + continue + + for path in keys: + del self._cache[path] + return keys + def _markdirty(self, path, exists, data=None, date=None, flags=''): self._cache[path] = { 'exists': exists,