This is an archive of the discontinued Mercurial Phabricator instance.

Status	Author	Revision
Accepted	kevincox	D3454 Assign to result from block.
Accepted	indygreg	D3447 rust: make exit handling consistent with `hg`
Accepted	indygreg	D3446 dispatch: move sys.exit() to caller (API)
Needs Revision	indygreg	D3445 dispatch: validate return type from dispatch() return value
Closed	indygreg	D3537 dispatch: minor code refactor
Needs Revision	indygreg	D3444 tests: comprehensively test exit handling

Diff 8653

	raise exc			raise exc

	def run():			def run():
	"run the command in sys.argv"			"run the command in sys.argv"
	initstdio()			initstdio()
	req = request(pycompat.sysargv[1:])			req = request(pycompat.sysargv[1:])
	err = None			err = None
	try:			try:
	status = (dispatch(req) or 0)			status = dispatch(req) or 0
	except error.StdioError as e:			except error.StdioError as e:
	err = e			err = e
	status = -1			status = -1

				# In all cases we try to flush stdio streams.
	if util.safehasattr(req.ui, 'fout'):			if util.safehasattr(req.ui, 'fout'):
	try:			try:
	req.ui.fout.flush()			req.ui.fout.flush()
	except IOError as e:			except IOError as e:
	err = e			err = e
	status = -1			status = -1

	if util.safehasattr(req.ui, 'ferr'):			if util.safehasattr(req.ui, 'ferr'):
	try:			try:
	if err is not None and err.errno != errno.EPIPE:			if err is not None and err.errno != errno.EPIPE:
	req.ui.ferr.write('abort: %s\n' %			req.ui.ferr.write('abort: %s\n' %
	encoding.strtolocal(err.strerror))			encoding.strtolocal(err.strerror))
	req.ui.ferr.flush()			req.ui.ferr.flush()
	# There's not much we can do about an I/O error here. So (possibly)			# There's not much we can do about an I/O error here. So (possibly)
	# change the status code and move on.			# change the status code and move on.