[Commented On] D11495: narrow: make hg manifest only print things inside the narrowspec

[spectral (Kyle Lippincott)]

spectral added a comment.


  In D11495#176286 <https://phab.mercurial-scm.org/D11495#176286>, @martinvonz wrote:
  
  > In D11495#176284 <https://phab.mercurial-scm.org/D11495#176284>, @durin42 wrote:
  >
  >> I think that's a buggy test? @martinvonz and @spectral might have an opinion.
  >
  > I think I did it that way because I felt that `hg manifest` is a debug command so it might be useful to have the information about excluded trees there.
  
  This comment highlights when it's useful to have `hg manifest` produce the more complete information: when using tree manifests :), otherwise `hg manifest` and `hg files` seem roughly equivalent to me (except that `hg manifest` can be coerced to print permissions/flags/hashes, and files can't. I think I'd slightly prefer if we could have a flag, possibly one that's only available when the narrow extension is loaded, that controls this behavior? I don't have any strong preferences for what the default is.
  
  > I never use it, however, so I don't mind if you change it.
  > PS. I wonder if @charlesetc actually wanted to use `hg files`. If not, what's the reason for using `hg manifest`?

REPOSITORY
  rHG Mercurial

CHANGES SINCE LAST ACTION
  https://phab.mercurial-scm.org/D11495/new/

REVISION DETAIL
  https://phab.mercurial-scm.org/D11495

To: charlesetc, durin42, #hg-reviewers, Alphare
Cc: martinvonz, spectral, Alphare, mercurial-patches
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.mercurial-scm.org/pipermail/mercurial-patches/attachments/20210928/845920b4/attachment-0002.html>