Mercurial > hg-stable
changeset 32416:c942c83ac2ec
debugcommands: use temporary dict for its command table
Instead, load the table by commands.py so the debug commands should always
be populated. The table in debugcommands.py is unnamed so extension authors
wouldn't be confused to wrap debugcommands.table in place of commands.table.
author | Yuya Nishihara <yuya@tcha.org> |
---|---|
date | Thu, 04 May 2017 17:13:12 +0900 |
parents | b16cb0a27377 |
children | 7d0c69505a66 |
files | hgext/largefiles/uisetup.py mercurial/commands.py mercurial/debugcommands.py mercurial/dispatch.py |
diffstat | 4 files changed, 3 insertions(+), 15 deletions(-) [+] |
line wrap: on
line diff
--- a/hgext/largefiles/uisetup.py Sun May 14 16:42:16 2017 +0900 +++ b/hgext/largefiles/uisetup.py Thu May 04 17:13:12 2017 +0900 @@ -21,7 +21,6 @@ cmdutil, commands, copies, - debugcommands, exchange, extensions, filemerge, @@ -40,11 +39,6 @@ ) def uisetup(ui): - # TODO: debugcommands should use a separate command table - # Side-effect of accessing is debugcommands module is guaranteed to be - # imported and commands.table is populated. - debugcommands.command - # Disable auto-status for some commands which assume that all # files in the result are under Mercurial's control
--- a/mercurial/commands.py Sun May 14 16:42:16 2017 +0900 +++ b/mercurial/commands.py Thu May 04 17:13:12 2017 +0900 @@ -26,6 +26,7 @@ changegroup, cmdutil, copies, + debugcommands as debugcommandsmod, destutil, dirstateguard, discovery, @@ -59,6 +60,7 @@ release = lockmod.release table = {} +table.update(debugcommandsmod.command._table) command = registrar.command(table)
--- a/mercurial/debugcommands.py Sun May 14 16:42:16 2017 +0900 +++ b/mercurial/debugcommands.py Thu May 04 17:13:12 2017 +0900 @@ -32,7 +32,6 @@ changegroup, cmdutil, color, - commands, context, dagparser, dagutil, @@ -71,9 +70,7 @@ release = lockmod.release -# We reuse the command table from commands because it is easier than -# teaching dispatch about multiple tables. -command = registrar.command(commands.table) +command = registrar.command() @command('debugancestor', [], _('[INDEX] REV1 REV2'), optionalrepo=True) def debugancestor(ui, repo, *args):
--- a/mercurial/dispatch.py Sun May 14 16:42:16 2017 +0900 +++ b/mercurial/dispatch.py Thu May 04 17:13:12 2017 +0900 @@ -25,7 +25,6 @@ cmdutil, color, commands, - debugcommands, demandimport, encoding, error, @@ -750,10 +749,6 @@ rpath = _earlygetopt(["-R", "--repository", "--repo"], args) path, lui = _getlocal(ui, rpath) - # Side-effect of accessing is debugcommands module is guaranteed to be - # imported and commands.table is populated. - debugcommands.command - uis = {ui, lui} if req.repo: