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