On 2/13/22 23:37, Pierre-Yves David wrote:
Hello everyone
The 6.1 freeze is looming and I would like to discuss finalizing the
`dirstate-tracked-key` feature.
The main remaining question if the configuration option. That already
sparked some discussion in the initial series but was a secondary
issue as long as the feature experimental.
So after thinking more about this I don't think the `version` concerns
warrant and special case. And I advocate for the usual
`format.use-dirstate-tracked-key=yes` option. If needed, the version
configuration might be dealt with by a secondary option: probably
`format.use-dirstate-tracked-key:version`. It might be smart to
introduce it already, to properly bark when the value is not "1".
However this might seems premature.
What do you think ?
I think doing a tiny bit of work to have it forward compatible is worth
it, we'd want an older client to bark if an unknown version were to show
up. The exact wording of the option doesn't matter to me much.
Cheers
PS: I am adding Kyle in CC as this feature is the related to a need he
expressed with dirstate and concurrency issue Google had. This is not
exactly what they wanted, but close enough to make sure they are aware
of it. It might even fit there underlying needs ? I though I did the
CC on the initial series but I apparently did not.
___
Mercurial-devel mailing list
Mercurial-devel@mercurial-scm.org
https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel