changeset 11695:ee8f36a6c766

alias: improved diagnostic when arguments include --cwd, etc. The logic pre-emptively checks for -R, --repo, --repository and --cwd in order to give the user a more helpful error message. In addition, each option is handled invididually, which avoids listing them all in the error. Before: % hg --config alias.broken='push --cwd /dev/null' broken abort: Option --cwd may not be abbreviated! After: % hg --config alias.broken='push --cwd /dev/null' broken error in definition for alias 'broken': --cwd may only be given on the command line
author Dan Villiom Podlaski Christiansen <danchr@gmail.com>
date Tue, 27 Jul 2010 14:43:40 +0200
parents bf49d48e4602
children 708291e9389c
files mercurial/dispatch.py tests/test-alias tests/test-alias.out
diffstat 3 files changed, 35 insertions(+), 0 deletions(-) [+]
line wrap: on
line diff
--- a/mercurial/dispatch.py	Tue Jul 27 20:50:09 2010 +0530
+++ b/mercurial/dispatch.py	Tue Jul 27 14:43:40 2010 +0200
@@ -223,6 +223,18 @@
         cmd = args.pop(0)
         args = map(util.expandpath, args)
 
+        for invalidarg in ("--cwd", "-R", "--repository", "--repo"):
+            if _earlygetopt([invalidarg], args):
+                def fn(ui, *args):
+                    ui.warn(_("error in definition for alias '%s': %s may only "
+                              "be given on the command line\n")
+                            % (self.name, invalidarg))
+                    return 1
+
+                self.fn = fn
+                self.badalias = True
+                return
+
         try:
             tableentry = cmdutil.findcmd(cmd, cmdtable, False)[1]
             if len(tableentry) > 2:
--- a/tests/test-alias	Tue Jul 27 20:50:09 2010 +0530
+++ b/tests/test-alias	Tue Jul 27 14:43:40 2010 +0200
@@ -8,6 +8,10 @@
 ambiguous = s
 recursive = recursive
 nodefinition =
+no--cwd = status --cwd elsewhere
+no-R = status -R elsewhere
+no--repo = status --repo elsewhere
+no--repository = status --repository elsewhere
 mylog = log
 lognull = log -r null
 shortlog = log --template '{rev} {node|short} | {date|isodate}\n'
@@ -42,6 +46,16 @@
 hg nodef
 hg help nodef
 
+echo '% invalid options'
+hg no--cwd
+hg help no--cwd
+hg no-R
+hg help no-R
+hg no--repo
+hg help no--repo
+hg no--repository
+hg help no--repository
+
 cd alias
 
 echo '% no usage'
--- a/tests/test-alias.out	Tue Jul 27 20:50:09 2010 +0530
+++ b/tests/test-alias.out	Tue Jul 27 14:43:40 2010 +0200
@@ -11,6 +11,15 @@
 % no definition
 no definition for alias 'nodefinition'
 no definition for alias 'nodefinition'
+% invalid options
+error in definition for alias 'no--cwd': --cwd may only be given on the command line
+error in definition for alias 'no--cwd': --cwd may only be given on the command line
+error in definition for alias 'no-R': -R may only be given on the command line
+error in definition for alias 'no-R': -R may only be given on the command line
+error in definition for alias 'no--repo': --repo may only be given on the command line
+error in definition for alias 'no--repo': --repo may only be given on the command line
+error in definition for alias 'no--repository': --repository may only be given on the command line
+error in definition for alias 'no--repository': --repository may only be given on the command line
 % no usage
 no rollback information available
 adding foo