This is an archive of the discontinued Mercurial Phabricator instance.

Status	Author	Revision
Closed	durin42	D3529 tests: use stringutil.pprint instead of custom dumper in test-hgweb-auth.py
Closed	durin42	D3528 tests: port test-simplekeyvaluefile.py to Python 3
Closed	durin42	D3527 tests: port test-extensions-wrapfunction to Python 3
Closed	durin42	D3526 tests: port test-ui-config to Python 3
Closed	durin42	D3525 config: fix py3 backslash escaping bug in parser caught by tests
Closed	durin42	D3524 tests: port test-symlink-os-yes-fs-no.py to Python 3
Closed	durin42	D3523 posix: use inst.errno instead of inst[0] on OSError instances
Closed	durin42	D3522 tests: port test-filelog.py to Python 3
Closed	durin42	D3521 stringutil: teach pprint about tuples

Diff 8617

	else:			else:
	try:			try:
	os.rename(name, checklink)			os.rename(name, checklink)
	except OSError:			except OSError:
	unlink(name)			unlink(name)
	return True			return True
	except OSError as inst:			except OSError as inst:
	# link creation might race, try again			# link creation might race, try again
	if inst[0] == errno.EEXIST:			if inst.errno == errno.EEXIST:
	continue			continue
	raise			raise
	finally:			finally:
	if fd is not None:			if fd is not None:
	fd.close()			fd.close()
	except AttributeError:			except AttributeError:
	return False			return False
	except OSError as inst:			except OSError as inst:
	# sshfs might report failure while successfully creating the link			# sshfs might report failure while successfully creating the link
	if inst[0] == errno.EIO and os.path.exists(name):			if inst.errno == errno.EIO and os.path.exists(name):
	unlink(name)			unlink(name)
	return False			return False

	def checkosfilename(path):			def checkosfilename(path):
	'''Check that the base-relative path is a valid filename on this platform.			'''Check that the base-relative path is a valid filename on this platform.
	Returns None if the path is ok, or a UI string describing the problem.'''			Returns None if the path is ok, or a UI string describing the problem.'''
	return None # on posix platforms, every path is ok			return None # on posix platforms, every path is ok