Mercurial > hg-stable
changeset 45900:600aec73f309
errors: format "abort: " text in a new Abort.format() method
This remove some duplication we had.
Differential Revision: https://phab.mercurial-scm.org/D9348
| author | Martin von Zweigbergk <martinvonz@google.com> |
|---|---|
| date | Fri, 20 Nov 2020 09:17:38 -0800 |
| parents | 98399dd1b96c |
| children | 18489e26d9a0 |
| files | mercurial/chgserver.py mercurial/dispatch.py mercurial/error.py mercurial/scmutil.py mercurial/wireprotov1server.py |
| diffstat | 5 files changed, 12 insertions(+), 12 deletions(-) [+] |
line wrap: on
line diff
--- a/mercurial/chgserver.py Fri Nov 20 08:51:45 2020 -0800 +++ b/mercurial/chgserver.py Fri Nov 20 09:17:38 2020 -0800 @@ -512,9 +512,7 @@ self.cresult.write(b'exit 255') return except error.Abort as inst: - self.ui.error(_(b"abort: %s\n") % inst.message) - if inst.hint: - self.ui.error(_(b"(%s)\n") % inst.hint) + self.ui.error(inst.format()) self.ui.flush() self.cresult.write(b'exit 255') return
--- a/mercurial/dispatch.py Fri Nov 20 08:51:45 2020 -0800 +++ b/mercurial/dispatch.py Fri Nov 20 09:17:38 2020 -0800 @@ -256,9 +256,7 @@ if req.fmsg: req.ui.fmsg = req.fmsg except error.Abort as inst: - ferr.write(_(b"abort: %s\n") % inst.message) - if inst.hint: - ferr.write(_(b"(%s)\n") % inst.hint) + ferr.write(inst.format()) return -1 except error.ParseError as inst: ferr.write(inst.format())
--- a/mercurial/error.py Fri Nov 20 08:51:45 2020 -0800 +++ b/mercurial/error.py Fri Nov 20 09:17:38 2020 -0800 @@ -182,6 +182,14 @@ # may raise another exception. return pycompat.sysstr(self.__bytes__()) + def format(self): + from .i18n import _ + + message = _(b"abort: %s\n") % self.message + if self.hint: + message += _(b"(%s)\n") % self.hint + return message + class InputError(Abort): """Indicates that the user made an error in their input.
--- a/mercurial/scmutil.py Fri Nov 20 08:51:45 2020 -0800 +++ b/mercurial/scmutil.py Fri Nov 20 09:17:38 2020 -0800 @@ -230,9 +230,7 @@ detailed_exit_code = 30 elif isinstance(inst, error.CanceledError): detailed_exit_code = 250 - ui.error(_(b"abort: %s\n") % inst.message) - if inst.hint: - ui.error(_(b"(%s)\n") % inst.hint) + ui.error(inst.format()) except error.WorkerError as inst: # Don't print a message -- the worker already should have return inst.status_code
--- a/mercurial/wireprotov1server.py Fri Nov 20 08:51:45 2020 -0800 +++ b/mercurial/wireprotov1server.py Fri Nov 20 09:17:38 2020 -0800 @@ -685,9 +685,7 @@ # We did not change it to minimise code change. # This need to be moved to something proper. # Feel free to do it. - procutil.stderr.write(b"abort: %s\n" % exc.message) - if exc.hint is not None: - procutil.stderr.write(b"(%s)\n" % exc.hint) + procutil.stderr.write(exc.format()) procutil.stderr.flush() return wireprototypes.pushres( 0, output.getvalue() if output else b''