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.
CHANGES SINCE LAST ACTION
To: valentin.gatienbaron, #hg-reviewers, mharbison72
Cc: JordiGH, yuja, mharbison72, martinvonz, mjpieters, mercurial-devel
More information about the Mercurial-devel