D6613: commit: improve the files field of changelog for merges

mharbison72 (Matt Harbison) phabricator at mercurial-scm.org
Tue Jul 16 21:46:40 EDT 2019


mharbison72 added a comment.
mharbison72 accepted this revision.


  In D6613#97136 <https://phab.mercurial-scm.org/D6613#97136>, @valentin.gatienbaron wrote:
  
  > @mharbison72  I dealt with the problem of convert in the previous commit. I didn't add a config option to disable the new code path, because it's really awkward to document, and I think it would be annoying to use if I made more fixes (the other case where files are spuriously listed, involving exec bits, probably has little impact on repo size/pull performance but it may be worth fixing anyway because it causes confusion). But if there are downsides to what I did, the config option is still on the table.
  
  I'm still fuzzy on how all of the manifest stuff works in general.  But it looks like the previous patch addresses my convert concerns, so I'm OK with not having a config option to disable this.

REPOSITORY
  rHG Mercurial

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

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

To: valentin.gatienbaron, #hg-reviewers, mharbison72
Cc: JordiGH, yuja, mharbison72, martinvonz, mjpieters, mercurial-devel


More information about the Mercurial-devel mailing list