D1336: HG: hg rm -A option prints the message of every file in the repo
mharbison72 (Matt Harbison)
phabricator at mercurial-scm.org
Wed Nov 8 22:48:59 EST 2017
mharbison72 added a comment.
Thank you for the proposed change. Does --quiet work for you?
I don't think I necessarily agree with the premise here. I think the only way every file in the repo is printed is if you delete every file first. Many file operations like add, remove/forget, and addremove all follow the style of quiet if files are explicitly named, and verbose for inexact matches. I've come to rely on that to catch mistakes when I'm not explicit about file paths. I'm not sure if it's worse to dump a ton of output to the screen, or not consistently follow the rules.
I'm sure others will chime in, but in the meantime, there's an inline comment too.
INLINE COMMENTS
> cmdutil.py:2970
> list = modified + deleted + clean + added
> - elif after:
> + elif after or ui.verbose:
> list = deleted
Should ui.verbose be nested inside the elif instead? The -v without -A case would have taken the 'else' case before, so this change seems to affect other things unintentionally.
REPOSITORY
rHG Mercurial
REVISION DETAIL
https://phab.mercurial-scm.org/D1336
To: pavanpc, #hg-reviewers
Cc: mharbison72, mercurial-devel
More information about the Mercurial-devel
mailing list