Mercurial > hg
comparison hgext/journal.py @ 33436:9bb4decd43b0
repovfs: add a ward to check if locks are properly taken
When the appropriate developer warnings are enabled, We wrap 'repo.vfs.audit' to
check for locks when accessing file in '.hg' for writing. Another changeset will
add a 'ward' for the store vfs (svfs).
This check system has caught a handful of locking issues that have been fixed
in previous series (mostly in 4.0). I expect another batch to be caught in third
party extensions.
We introduce two real exceptions from extensions 'blackbox.log' (because a lot of
read-only operations add entry to it), and 'last-email.txt' (because 'hg email'
is currently a read only operation and there is value to keep it this way).
In addition we are currently allowing bisect to operate outside of the lock
because the current code is a bit hard to get properly locked for now. Multiple
clean up have been made but there is still a couple of them to do and the freeze
is coming.
author | Boris Feld <boris.feld@octobus.net> |
---|---|
date | Tue, 11 Jul 2017 12:38:17 +0200 |
parents | bc1f373cf3f6 |
children | 0407a51b9d8c |
comparison
equal
deleted
inserted
replaced
33435:456626e9c3d1 | 33436:9bb4decd43b0 |
---|---|
67 extensions.wrapfunction(hg, 'copystore', unsharejournal) | 67 extensions.wrapfunction(hg, 'copystore', unsharejournal) |
68 | 68 |
69 def reposetup(ui, repo): | 69 def reposetup(ui, repo): |
70 if repo.local(): | 70 if repo.local(): |
71 repo.journal = journalstorage(repo) | 71 repo.journal = journalstorage(repo) |
72 repo._wlockfreeprefix.add('namejournal') | |
72 | 73 |
73 dirstate, cached = localrepo.isfilecached(repo, 'dirstate') | 74 dirstate, cached = localrepo.isfilecached(repo, 'dirstate') |
74 if cached: | 75 if cached: |
75 # already instantiated dirstate isn't yet marked as | 76 # already instantiated dirstate isn't yet marked as |
76 # "journal"-ing, even though repo.dirstate() was already | 77 # "journal"-ing, even though repo.dirstate() was already |