Patchwork D1841: exchange: use context manager for locks and transaction in unbundle()

login
register
mail settings
Submitter phabricator
Date Jan. 10, 2018, 11:03 p.m.
Message ID <bfef808372c04e9c3869f7e9a451f823@localhost.localdomain>
Download mbox | patch
Permalink /patch/26654/
State Not Applicable
Headers show

Comments

phabricator - Jan. 10, 2018, 11:03 p.m.
This revision was automatically updated to reflect the committed changes.
Closed by commit rHG86f9aabed67b: exchange: use context manager for locks and transaction in unbundle() (authored by martinvonz, committed by ).

REPOSITORY
  rHG Mercurial

CHANGES SINCE LAST UPDATE
  https://phab.mercurial-scm.org/D1841?vs=4761&id=4771

REVISION DETAIL
  https://phab.mercurial-scm.org/D1841

AFFECTED FILES
  mercurial/exchange.py

CHANGE DETAILS




To: martinvonz, #hg-reviewers, durin42
Cc: mercurial-devel

Patch

diff --git a/mercurial/exchange.py b/mercurial/exchange.py
--- a/mercurial/exchange.py
+++ b/mercurial/exchange.py
@@ -1345,11 +1345,8 @@ 
                     " %s") % (', '.join(sorted(missing)))
             raise error.Abort(msg)
 
-    wlock = lock = None
-    try:
-        wlock = pullop.repo.wlock()
-        lock = pullop.repo.lock()
-        pullop.trmanager = transactionmanager(repo, 'pull', remote.url())
+    pullop.trmanager = transactionmanager(repo, 'pull', remote.url())
+    with repo.wlock(), repo.lock(), pullop.trmanager:
         # This should ideally be in _pullbundle2(). However, it needs to run
         # before discovery to avoid extra work.
         _maybeapplyclonebundle(pullop)
@@ -1361,9 +1358,6 @@ 
         _pullphase(pullop)
         _pullbookmarks(pullop)
         _pullobsolete(pullop)
-        pullop.trmanager.close()
-    finally:
-        lockmod.release(pullop.trmanager, lock, wlock)
 
     # storing remotenames
     if repo.ui.configbool('experimental', 'remotenames'):