D6183: copies: add config option for writing copy metadata to file and/or changset
gracinet (Georges Racinet)
phabricator at mercurial-scm.org
Thu Apr 11 05:22:51 EDT 2019
gracinet added a comment.
We had half an hour of direct chat about this yesterday, with @martinvonz and @marmoute. Here's a summary
- The architecture of the proposed change makes the storage and conveying of the information orthogonal to its usage. Therefore, it can be built upon to introduce other storage strategies, such as the caching system suggested by @marmoute.
- The documentation for the new config option should clearly state that `changeset-only` is applicable only in cases where all the peers enable it and prior changesets hashes don't matter. I'm not 100% sure of the exact technical condition at this point, but I suppose someone can find a simple sentence that would be correct enough.
- Octobus hopes that the caching strategy could eventually become useful for the general public.
- @martinvonz helped us understand his implementation by answering further technical questions by @marmoute.
Overall, my feeling is that it's been a very productive talk and that we have a clear path forward.
To: martinvonz, #hg-reviewers
Cc: pulkit, gracinet, marmoute, mercurial-devel
More information about the Mercurial-devel