Patchwork D517: journal: do not use atomictemp

login
register
mail settings
Submitter phabricator
Date Aug. 25, 2017, 3:25 p.m.
Message ID <e2b8eeef987b733d80feb4caa0cda2ba@localhost.localdomain>
Download mbox | patch
Permalink /patch/23334/
State Not Applicable
Headers show

Comments

phabricator - Aug. 25, 2017, 3:25 p.m.
This revision was automatically updated to reflect the committed changes.
Closed by commit rHGe6d421566906: journal: do not use atomictemp (issue5338) (authored by quark).

REPOSITORY
  rHG Mercurial

CHANGES SINCE LAST UPDATE
  https://phab.mercurial-scm.org/D517?vs=1281&id=1293

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

AFFECTED FILES
  hgext/journal.py

CHANGE DETAILS




To: quark, #hg-reviewers, yuja
Cc: yuja, mercurial-devel

Patch

diff --git a/hgext/journal.py b/hgext/journal.py
--- a/hgext/journal.py
+++ b/hgext/journal.py
@@ -342,7 +342,7 @@ 
         with self.jlock(vfs):
             version = None
             # open file in amend mode to ensure it is created if missing
-            with vfs('namejournal', mode='a+b', atomictemp=True) as f:
+            with vfs('namejournal', mode='a+b') as f:
                 f.seek(0, os.SEEK_SET)
                 # Read just enough bytes to get a version number (up to 2
                 # digits plus separator)