This is an archive of the discontinued Mercurial Phabricator instance.

Differential D5454

help: show "[no-]" only for default-on Flags
ClosedPublic

Authored by martinvonz on Dec 19 2018, 11:50 AM.

Details

Reviewers
durin42
av6
Group Reviewers
hg-reviewers
Commits
rHGfcc0a7ac9ebd: help: show "[no-]" only for default-on Flags
Summary

As Anton (av6) pointed out, the "[no-]" is confusing for action flags
like hg bookmark --delete. We could come up with a way of indicating
which flags are action flags (e.g. use None for the default value
instead of False). However, it's probably also unlikely that users
will want to negate even non-action flags like --hidden.

One of the more common flags where the "[no-]" prefix would be useful
is hg evolve --update. The reason it's helpful there is that it
defaults to on. So I think we can simply include "[no-]" only for
flags that are on by default (and thus require the user to add the
"[no-]" for the option to have any effect).

Note that there are use cases for negating flags that already off by
default. For example, you may have an alias for `hg log -G --hidden -T
foo` and now want to pass "--no-hidden" to that alias. However, I
think that users who want that are likely to be advanced enough that
they've already learnt about the "no-" prefix by seeing it somewhere
else.

Diff Detail

Repository
rHG Mercurial
Lint
Lint Skipped
Unit
Unit Tests Skipped

Event Timeline

martinvonz created this revision.Dec 19 2018, 11:50 AM
martinvonz edited the summary of this revision. (Show Details)Dec 19 2018, 12:42 PM
martinvonz retitled this revision from help: hide default value for boolean flag and show "[no-]" only for default-on to help: "[no-]" only for default-on Flags.
martinvonz added a child revision: D5455: help: hide default value for default-off flags.
martinvonz updated this revision to Diff 12902.
martinvonz retitled this revision from help: "[no-]" only for default-on Flags to help: show "[no-]" only for default-on Flags.Dec 19 2018, 12:46 PM
av6 accepted this revision.Dec 20 2018, 6:21 AM
This revision is now accepted and ready to land.Dec 20 2018, 6:21 AM
Closed by commit rHGfcc0a7ac9ebd: help: show "[no-]" only for default-on Flags (authored by martinvonz). · Explain WhyDec 20 2018, 7:02 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 12901

mercurial/help.py

if not verbose and any(w in desc for w in _exclkeywords): if not verbose and any(w in desc for w in _exclkeywords):
continue continue
so = '' so = ''
if shortopt: if shortopt:
so = '-' + shortopt so = '-' + shortopt
lo = '--' + longopt lo = '--' + longopt
if isinstance(default, bool): if default is True:
lo = '--[no-]' + longopt lo = '--[no-]' + longopt
if isinstance(default, fancyopts.customopt): if isinstance(default, fancyopts.customopt):
default = default.getdefaultvalue() default = default.getdefaultvalue()
if (default and not callable(default)) or default is False: if (default and not callable(default)):
# default is of unknown type, and in Python 2 we abused # default is of unknown type, and in Python 2 we abused
# the %s-shows-repr property to handle integers etc. To # the %s-shows-repr property to handle integers etc. To
# match that behavior on Python 3, we do str(default) and # match that behavior on Python 3, we do str(default) and
# then convert it to bytes. # then convert it to bytes.
defaultstr = pycompat.bytestr(default) if default is not True:
if isinstance(default, bool): desc += _(" (default: %s)") % pycompat.bytestr(default)
defaultstr = _("on") if default else _("off")
desc += _(" (default: %s)") % defaultstr
if isinstance(default, list): if isinstance(default, list):
lo += " %s [+]" % optlabel lo += " %s [+]" % optlabel
multioccur = True multioccur = True
elif (default is not None) and not isinstance(default, bool): elif (default is not None) and not isinstance(default, bool):
lo += " %s" % optlabel lo += " %s" % optlabel
data.append((so, lo, desc)) data.append((so, lo, desc))

tests/test-extension.t

--debugger start debugger --debugger start debugger
--encoding ENCODE set the charset encoding (default: ascii) --encoding ENCODE set the charset encoding (default: ascii)
--encodingmode MODE set the charset encoding mode (default: strict) --encodingmode MODE set the charset encoding mode (default: strict)
--traceback always print a traceback on exception --traceback always print a traceback on exception
--time time how long the command takes --time time how long the command takes
--profile print command execution profile --profile print command execution profile
--version output version information and exit --version output version information and exit
-h --help display help and exit -h --help display help and exit
--[no-]hidden consider hidden changesets (default: off) --hidden consider hidden changesets
--pager TYPE when to paginate (boolean, always, auto, or never) --pager TYPE when to paginate (boolean, always, auto, or never)
(default: auto) (default: auto)
--debugger start debugger --debugger start debugger
--encoding ENCODE set the charset encoding (default: ascii) --encoding ENCODE set the charset encoding (default: ascii)
--encodingmode MODE set the charset encoding mode (default: strict) --encodingmode MODE set the charset encoding mode (default: strict)
--traceback always print a traceback on exception --traceback always print a traceback on exception
--time time how long the command takes --time time how long the command takes
--profile print command execution profile --profile print command execution profile
--version output version information and exit --version output version information and exit
-h --help display help and exit -h --help display help and exit
--[no-]hidden consider hidden changesets (default: off) --hidden consider hidden changesets
--pager TYPE when to paginate (boolean, always, auto, or never) --pager TYPE when to paginate (boolean, always, auto, or never)
(default: auto) (default: auto)
$ echo 'debugextension = !' >> $HGRCPATH $ echo 'debugextension = !' >> $HGRCPATH
--debugger start debugger --debugger start debugger
--encoding ENCODE set the charset encoding (default: ascii) --encoding ENCODE set the charset encoding (default: ascii)
--encodingmode MODE set the charset encoding mode (default: strict) --encodingmode MODE set the charset encoding mode (default: strict)
--traceback always print a traceback on exception --traceback always print a traceback on exception
--time time how long the command takes --time time how long the command takes
--profile print command execution profile --profile print command execution profile
--version output version information and exit --version output version information and exit
-h --help display help and exit -h --help display help and exit
--[no-]hidden consider hidden changesets (default: off) --hidden consider hidden changesets
--pager TYPE when to paginate (boolean, always, auto, or never) --pager TYPE when to paginate (boolean, always, auto, or never)
(default: auto) (default: auto)
Make sure that single '-v' option shows help and built-ins only for 'dodo' command Make sure that single '-v' option shows help and built-ins only for 'dodo' command
$ hg help -v dodo $ hg help -v dodo
hg dodo hg dodo
Does nothing Does nothing
--debugger start debugger --debugger start debugger
--encoding ENCODE set the charset encoding (default: ascii) --encoding ENCODE set the charset encoding (default: ascii)
--encodingmode MODE set the charset encoding mode (default: strict) --encodingmode MODE set the charset encoding mode (default: strict)
--traceback always print a traceback on exception --traceback always print a traceback on exception
--time time how long the command takes --time time how long the command takes
--profile print command execution profile --profile print command execution profile
--version output version information and exit --version output version information and exit
-h --help display help and exit -h --help display help and exit
--[no-]hidden consider hidden changesets (default: off) --hidden consider hidden changesets
--pager TYPE when to paginate (boolean, always, auto, or never) --pager TYPE when to paginate (boolean, always, auto, or never)
(default: auto) (default: auto)
In case when extension name doesn't match any of its commands, In case when extension name doesn't match any of its commands,
help message should ask for '-v' to get list of built-in aliases help message should ask for '-v' to get list of built-in aliases
along with extension help along with extension help
$ cat > $TESTTMP/d/dudu.py <<EOF $ cat > $TESTTMP/d/dudu.py <<EOF
> """ > """
--debugger start debugger --debugger start debugger
--encoding ENCODE set the charset encoding (default: ascii) --encoding ENCODE set the charset encoding (default: ascii)
--encodingmode MODE set the charset encoding mode (default: strict) --encodingmode MODE set the charset encoding mode (default: strict)
--traceback always print a traceback on exception --traceback always print a traceback on exception
--time time how long the command takes --time time how long the command takes
--profile print command execution profile --profile print command execution profile
--version output version information and exit --version output version information and exit
-h --help display help and exit -h --help display help and exit
--[no-]hidden consider hidden changesets (default: off) --hidden consider hidden changesets
--pager TYPE when to paginate (boolean, always, auto, or never) --pager TYPE when to paginate (boolean, always, auto, or never)
(default: auto) (default: auto)
$ hg help -v -e dudu $ hg help -v -e dudu
dudu extension - dudu extension -
This is an awesome 'dudu' extension. It does something and also writes 'Beep This is an awesome 'dudu' extension. It does something and also writes 'Beep
beep' beep'
--debugger start debugger --debugger start debugger
--encoding ENCODE set the charset encoding (default: ascii) --encoding ENCODE set the charset encoding (default: ascii)
--encodingmode MODE set the charset encoding mode (default: strict) --encodingmode MODE set the charset encoding mode (default: strict)
--traceback always print a traceback on exception --traceback always print a traceback on exception
--time time how long the command takes --time time how long the command takes
--profile print command execution profile --profile print command execution profile
--version output version information and exit --version output version information and exit
-h --help display help and exit -h --help display help and exit
--[no-]hidden consider hidden changesets (default: off) --hidden consider hidden changesets
--pager TYPE when to paginate (boolean, always, auto, or never) --pager TYPE when to paginate (boolean, always, auto, or never)
(default: auto) (default: auto)
Disabled extension commands: Disabled extension commands:
$ ORGHGRCPATH=$HGRCPATH $ ORGHGRCPATH=$HGRCPATH
$ HGRCPATH= $ HGRCPATH=
$ export HGRCPATH $ export HGRCPATH

tests/test-fix.t

revisions are not forgotten in later ones. The --base flag can be used to revisions are not forgotten in later ones. The --base flag can be used to
override this default behavior, though it is not usually desirable to do override this default behavior, though it is not usually desirable to do
so. so.
(use 'hg help -e fix' to show help for the fix extension) (use 'hg help -e fix' to show help for the fix extension)
options ([+] can be repeated): options ([+] can be repeated):
--[no-]all fix all non-public non-obsolete revisions (default: --all fix all non-public non-obsolete revisions
off) --base REV [+] revisions to diff against (overrides automatic selection,
--base REV [+] revisions to diff against (overrides automatic and applies to every revision being fixed)
selection, and applies to every revision being fixed)
-r --rev REV [+] revisions to fix -r --rev REV [+] revisions to fix
-w --[no-]working-dir fix the working directory (default: off) -w --working-dir fix the working directory
--[no-]whole always fix every line of a file (default: off) --whole always fix every line of a file
(some details hidden, use --verbose to show complete help) (some details hidden, use --verbose to show complete help)
$ hg help -e fix $ hg help -e fix
fix extension - rewrite file content in changesets or working copy fix extension - rewrite file content in changesets or working copy
(EXPERIMENTAL) (EXPERIMENTAL)
Provides a command that runs configured tools on the contents of modified Provides a command that runs configured tools on the contents of modified

tests/test-help.t

--debugger start debugger --debugger start debugger
--encoding ENCODE set the charset encoding (default: ascii) --encoding ENCODE set the charset encoding (default: ascii)
--encodingmode MODE set the charset encoding mode (default: strict) --encodingmode MODE set the charset encoding mode (default: strict)
--traceback always print a traceback on exception --traceback always print a traceback on exception
--time time how long the command takes --time time how long the command takes
--profile print command execution profile --profile print command execution profile
--version output version information and exit --version output version information and exit
-h --help display help and exit -h --help display help and exit
--[no-]hidden consider hidden changesets (default: off) --hidden consider hidden changesets
--pager TYPE when to paginate (boolean, always, auto, or never) --pager TYPE when to paginate (boolean, always, auto, or never)
(default: auto) (default: auto)
(use 'hg help' for the full list of commands) (use 'hg help' for the full list of commands)
$ hg add -h $ hg add -h
hg add [OPTION]... [FILE]... hg add [OPTION]... [FILE]...
--debugger start debugger --debugger start debugger
--encoding ENCODE set the charset encoding (default: ascii) --encoding ENCODE set the charset encoding (default: ascii)
--encodingmode MODE set the charset encoding mode (default: strict) --encodingmode MODE set the charset encoding mode (default: strict)
--traceback always print a traceback on exception --traceback always print a traceback on exception
--time time how long the command takes --time time how long the command takes
--profile print command execution profile --profile print command execution profile
--version output version information and exit --version output version information and exit
-h --help display help and exit -h --help display help and exit
--[no-]hidden consider hidden changesets (default: off) --hidden consider hidden changesets
--pager TYPE when to paginate (boolean, always, auto, or never) --pager TYPE when to paginate (boolean, always, auto, or never)
(default: auto) (default: auto)
Test the textwidth config option Test the textwidth config option
$ hg root -h --config ui.textwidth=50 $ hg root -h --config ui.textwidth=50
hg root hg root
options: options:
--longdesc VALUE --longdesc VALUE
xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx
xxxxxxxxxxxxxxxxxxxxxxx (default: 3) xxxxxxxxxxxxxxxxxxxxxxx (default: 3)
-n -- normal desc -n -- normal desc
--newline VALUE line1 line2 --newline VALUE line1 line2
--[no-]default-off enable X (default: off) --default-off enable X
--[no-]default-on enable Y (default: on) --[no-]default-on enable Y
--callableopt VALUE adds foo --callableopt VALUE adds foo
--customopt VALUE adds bar --customopt VALUE adds bar
--customopt-withdefault VALUE adds bar (default: foo) --customopt-withdefault VALUE adds bar (default: foo)
(some details hidden, use --verbose to show complete help) (some details hidden, use --verbose to show complete help)
Test that default list of commands includes extension commands that have help, Test that default list of commands includes extension commands that have help,
but not those that don't, except in verbose mode, when a keyword is passed, or but not those that don't, except in verbose mode, when a keyword is passed, or
<td>print command execution profile</td></tr> <td>print command execution profile</td></tr>
<tr><td></td> <tr><td></td>
<td>--version</td> <td>--version</td>
<td>output version information and exit</td></tr> <td>output version information and exit</td></tr>
<tr><td>-h</td> <tr><td>-h</td>
<td>--help</td> <td>--help</td>
<td>display help and exit</td></tr> <td>display help and exit</td></tr>
<tr><td></td> <tr><td></td>
<td>--[no-]hidden</td> <td>--hidden</td>
<td>consider hidden changesets (default: off)</td></tr> <td>consider hidden changesets</td></tr>
<tr><td></td> <tr><td></td>
<td>--pager TYPE</td> <td>--pager TYPE</td>
<td>when to paginate (boolean, always, auto, or never) (default: auto)</td></tr> <td>when to paginate (boolean, always, auto, or never) (default: auto)</td></tr>
</table> </table>
</div> </div>
</div> </div>
</div> </div>
<td>print command execution profile</td></tr> <td>print command execution profile</td></tr>
<tr><td></td> <tr><td></td>
<td>--version</td> <td>--version</td>
<td>output version information and exit</td></tr> <td>output version information and exit</td></tr>
<tr><td>-h</td> <tr><td>-h</td>
<td>--help</td> <td>--help</td>
<td>display help and exit</td></tr> <td>display help and exit</td></tr>
<tr><td></td> <tr><td></td>
<td>--[no-]hidden</td> <td>--hidden</td>
<td>consider hidden changesets (default: off)</td></tr> <td>consider hidden changesets</td></tr>
<tr><td></td> <tr><td></td>
<td>--pager TYPE</td> <td>--pager TYPE</td>
<td>when to paginate (boolean, always, auto, or never) (default: auto)</td></tr> <td>when to paginate (boolean, always, auto, or never) (default: auto)</td></tr>
</table> </table>
</div> </div>
</div> </div>
</div> </div>

tests/test-narrow-trackedcmd.t

$ hg tracked --import-rules $ hg tracked --import-rules
hg tracked: option --import-rules requires argument hg tracked: option --import-rules requires argument
hg tracked [OPTIONS]... [REMOTE] hg tracked [OPTIONS]... [REMOTE]
show or change the current narrowspec show or change the current narrowspec
options ([+] can be repeated): options ([+] can be repeated):
--addinclude VALUE [+] new paths to include --addinclude VALUE [+] new paths to include
--removeinclude VALUE [+] old paths to no longer include --removeinclude VALUE [+] old paths to no longer include
--addexclude VALUE [+] new paths to exclude --addexclude VALUE [+] new paths to exclude
--import-rules VALUE import narrowspecs from a file --import-rules VALUE import narrowspecs from a file
--removeexclude VALUE [+] old paths to no longer exclude --removeexclude VALUE [+] old paths to no longer exclude
--[no-]clear whether to replace the existing --clear whether to replace the existing narrowspec
narrowspec (default: off) --force-delete-local-changes forces deletion of local changes when
--[no-]force-delete-local-changes forces deletion of local changes when narrowing
narrowing (default: off)
-e --ssh CMD specify ssh command to use -e --ssh CMD specify ssh command to use
--remotecmd CMD specify hg command to run on the remote --remotecmd CMD specify hg command to run on the remote side
side --insecure do not verify server certificate (ignoring
--insecure do not verify server certificate web.cacerts config)
(ignoring web.cacerts config)
(use 'hg tracked -h' to show more help) (use 'hg tracked -h' to show more help)
[255] [255]
$ hg tracked --import-rules doesnotexist $ hg tracked --import-rules doesnotexist
abort: cannot read narrowspecs from '$TESTTMP/narrow/doesnotexist': $ENOENT$ abort: cannot read narrowspecs from '$TESTTMP/narrow/doesnotexist': $ENOENT$
[255] [255]
$ cat > specs <<EOF $ cat > specs <<EOF

tests/test-shelve.t

options ([+] can be repeated): options ([+] can be repeated):
-A --addremove mark new/missing files as added/removed before -A --addremove mark new/missing files as added/removed before
shelving shelving
-u --unknown store unknown files in the shelve -u --unknown store unknown files in the shelve
--cleanup delete all shelved changes --cleanup delete all shelved changes
--date DATE shelve with the specified commit date --date DATE shelve with the specified commit date
-d --delete delete the named shelved change(s) -d --delete delete the named shelved change(s)
-e --[no-]edit invoke editor on commit messages (default: off) -e --edit invoke editor on commit messages
-l --list list current shelves -l --list list current shelves
-m --message TEXT use text as shelve message -m --message TEXT use text as shelve message
-n --name NAME use the given name for the shelved commit -n --name NAME use the given name for the shelved commit
-p --patch output patches for changes (provide the names of the -p --patch output patches for changes (provide the names of the
shelved changes as positional arguments) shelved changes as positional arguments)
-i --interactive interactive mode, only works while creating a shelve -i --interactive interactive mode, only works while creating a shelve
--stat output diffstat-style summary of changes (provide --stat output diffstat-style summary of changes (provide
the names of the shelved changes as positional the names of the shelved changes as positional

tests/test-uncommit.t

If no files are specified, the commit will be pruned, unless --keep is If no files are specified, the commit will be pruned, unless --keep is
given. given.
(use 'hg help -e uncommit' to show help for the uncommit extension) (use 'hg help -e uncommit' to show help for the uncommit extension)
options ([+] can be repeated): options ([+] can be repeated):
--[no-]keep allow an empty commit after uncommiting (default: --keep allow an empty commit after uncommiting
off)
-I --include PATTERN [+] include names matching the given patterns -I --include PATTERN [+] include names matching the given patterns
-X --exclude PATTERN [+] exclude names matching the given patterns -X --exclude PATTERN [+] exclude names matching the given patterns
(some details hidden, use --verbose to show complete help) (some details hidden, use --verbose to show complete help)
Uncommit with no commits should fail Uncommit with no commits should fail
$ hg uncommit $ hg uncommit