diff hgext/narrow/narrowcommands.py @ 41027:74e023f5a922

narrow: replace "ui.warn(); return 1" by "raise error.Abort()" This is the usual way of doing it and I don't see a reason to do it differently here. Differential Revision: https://phab.mercurial-scm.org/D5468
author Martin von Zweigbergk <martinvonz@google.com>
date Fri, 23 Nov 2018 22:12:04 -0800
parents 77173267878b
children 095a0a63ad88
line wrap: on
line diff
--- a/hgext/narrow/narrowcommands.py	Sun Dec 16 00:21:54 2018 -0500
+++ b/hgext/narrow/narrowcommands.py	Fri Nov 23 22:12:04 2018 -0800
@@ -365,15 +365,13 @@
     """
     opts = pycompat.byteskwargs(opts)
     if repository.NARROW_REQUIREMENT not in repo.requirements:
-        ui.warn(_('The narrow command is only supported on respositories cloned'
-                  ' with --narrow.\n'))
-        return 1
+        raise error.Abort(_('the narrow command is only supported on '
+                            'respositories cloned with --narrow'))
 
     # Before supporting, decide whether it "hg tracked --clear" should mean
     # tracking no paths or all paths.
     if opts['clear']:
-        ui.warn(_('The --clear option is not yet supported.\n'))
-        return 1
+        raise error.Abort(_('the --clear option is not yet supported'))
 
     # import rules from a file
     newrules = opts.get('import_rules')