Mercurial > hg-stable
comparison mercurial/posix.py @ 36954:0585337ea787
cleanup: fix some latent open(path).read() et al calls we previously missed
This pattern was banned by check-code way back in 1b4b82063ce2 (may of
2011), but due to a regular expression rewriting bug in check-code
these particular callsites were never detected. Python 3.7 caught the
bug, which then exposed these errors.
Differential Revision: https://phab.mercurial-scm.org/D2863
author | Augie Fackler <augie@google.com> |
---|---|
date | Wed, 14 Mar 2018 15:39:28 -0400 |
parents | ffa3026d4196 |
children | e24802ea8dbd |
comparison
equal
deleted
inserted
replaced
36953:3d0178bf1039 | 36954:0585337ea787 |
---|---|
262 else: | 262 else: |
263 # create a fixed file to link to; doesn't matter if it | 263 # create a fixed file to link to; doesn't matter if it |
264 # already exists. | 264 # already exists. |
265 target = 'checklink-target' | 265 target = 'checklink-target' |
266 try: | 266 try: |
267 open(os.path.join(cachedir, target), 'w').close() | 267 fullpath = os.path.join(cachedir, target) |
268 open(fullpath, 'w').close() | |
268 except IOError as inst: | 269 except IOError as inst: |
269 if inst[0] == errno.EACCES: | 270 if inst[0] == errno.EACCES: |
270 # If we can't write to cachedir, just pretend | 271 # If we can't write to cachedir, just pretend |
271 # that the fs is readonly and by association | 272 # that the fs is readonly and by association |
272 # that the fs won't support symlinks. This | 273 # that the fs won't support symlinks. This |