| Submitter | Prabhu GS |
|---|---|
| Date | July 22, 2014, 3:37 p.m. |
| Message ID | <f3157bfd4969cfec2c0f.1406043463@sparta.ap01.lucent.com> |
| Download | mbox | patch |
| Permalink | /patch/5193/ |
| State | Changes Requested |
| Headers | show |
Comments
Hi,
Yeah, the issue was with aliases. Since "diff".startswith("di") there was
no problem with aliasing di and diff. But this failed because "commit" does
not startwith "ci".
All tests are passing.
cheers,
Prabhu
On Tue, Jul 22, 2014 at 9:07 PM, Prabhu Gnana Sundar <pprabhugs@gmail.com>
wrote:
> # HG changeset patch
> # User Prabhu Gnana Sundar <pprabhugs@gmail.com>
> # Date 1406011473 -19800
> # Tue Jul 22 12:14:33 2014 +0530
> # Branch stable
> # Node ID f3157bfd4969cfec2c0f6577841feca6638823b7
> # Parent a5168eb9b2bc488f42809872d22321a9825bc738
> dispatch: fix so that --help more reliably shows command help (issue4240)
>
> After hg 3.0, "hg showconfig --help" was showing the configuration man page
> instead of the 'hg help -c showconfig' output.
>
> This patch fixes the 'hg showconfig --help' regression issue by removing an
> unwanted assignement of aliases[0] to cmd and shows the correct command
> in the first line of help description.
> Also, added a test case to test 'hg showconfig --help'.
>
> diff -r a5168eb9b2bc -r f3157bfd4969 mercurial/cmdutil.py
> --- a/mercurial/cmdutil.py Mon Jul 21 11:27:24 2014 -0700
> +++ b/mercurial/cmdutil.py Tue Jul 22 12:14:33 2014 +0530
> @@ -44,6 +44,9 @@
> found = a
> break
> if found is not None:
> + if found.find("config") != -1:
> + aliases = [alias for alias in aliases if alias != cmd]
> + aliases.insert(0, cmd)
> if aliases[0].startswith("debug") or
> found.startswith("debug"):
> debugchoice[found] = (aliases, table[e])
> else:
> diff -r a5168eb9b2bc -r f3157bfd4969 tests/test-help.t
> --- a/tests/test-help.t Mon Jul 21 11:27:24 2014 -0700
> +++ b/tests/test-help.t Tue Jul 22 12:14:33 2014 +0530
> @@ -2085,3 +2085,41 @@
> $ "$TESTDIR/killdaemons.py" $DAEMON_PIDS
>
> #endif
> +
> +
> +Test showconfig help
> +
> + $ hg showconfig --help
> + hg showconfig [-u] [NAME]...
> +
> + aliases: config, debugconfig
> +
> + show combined config settings from all hgrc files
> +
> + With no arguments, print names and values of all config items.
> +
> + With one argument of the form section.name, print just the value
> of that
> + config item.
> +
> + With multiple arguments, print names and values of all config items
> with
> + matching section names.
> +
> + With --edit, start an editor on the user-level config file. With
> --global,
> + edit the system-wide config file. With --local, edit the
> repository-level
> + config file.
> +
> + With --debug, the source (filename and line number) is printed for
> each
> + config item.
> +
> + See "hg help config" for more information about config files.
> +
> + Returns 0 on success.
> +
> + options:
> +
> + -u --untrusted show untrusted configuration options
> + -e --edit edit user config
> + -l --local edit repository config
> + -g --global edit global config
> +
> + use "hg -v help showconfig" to show the global options
>
Patch
diff -r a5168eb9b2bc -r f3157bfd4969 mercurial/cmdutil.py --- a/mercurial/cmdutil.py Mon Jul 21 11:27:24 2014 -0700 +++ b/mercurial/cmdutil.py Tue Jul 22 12:14:33 2014 +0530 @@ -44,6 +44,9 @@ found = a break if found is not None: + if found.find("config") != -1: + aliases = [alias for alias in aliases if alias != cmd] + aliases.insert(0, cmd) if aliases[0].startswith("debug") or found.startswith("debug"): debugchoice[found] = (aliases, table[e]) else: diff -r a5168eb9b2bc -r f3157bfd4969 tests/test-help.t --- a/tests/test-help.t Mon Jul 21 11:27:24 2014 -0700 +++ b/tests/test-help.t Tue Jul 22 12:14:33 2014 +0530 @@ -2085,3 +2085,41 @@ $ "$TESTDIR/killdaemons.py" $DAEMON_PIDS #endif + + +Test showconfig help + + $ hg showconfig --help + hg showconfig [-u] [NAME]... + + aliases: config, debugconfig + + show combined config settings from all hgrc files + + With no arguments, print names and values of all config items. + + With one argument of the form section.name, print just the value of that + config item. + + With multiple arguments, print names and values of all config items with + matching section names. + + With --edit, start an editor on the user-level config file. With --global, + edit the system-wide config file. With --local, edit the repository-level + config file. + + With --debug, the source (filename and line number) is printed for each + config item. + + See "hg help config" for more information about config files. + + Returns 0 on success. + + options: + + -u --untrusted show untrusted configuration options + -e --edit edit user config + -l --local edit repository config + -g --global edit global config + + use "hg -v help showconfig" to show the global options