On Mon, Feb 19, 2024 at 8:49 AM Julian Foad <jul...@foad.me.uk> wrote:
> Hello Subversion devs.
>
> I had a thought that, as the move-tracking experimental design I was
> working on a few years ago isn't heading anywhere, we should remove the
> associated code, as it is better to keep a cleaner codebase on trunk and
> releases.
>
> The associated code is almost entirely decoupled from the public
> interfaces. (There was one undocumented environment-variable: see in
> the patch below.) Therefore there seems to be no impact from removing
> it. I did one basic test run ('make svnserveautocheck') at r1915144 and
> all those tests pass.
>
> I am happy to commit this patch if the community agrees.
>
> It looks like this:
>
(snipped the patch)
I am okay with this (conceptually -- I read the patch but didn't dig
deeper).
More specifically, I am okay with removing it from trunk, but may I suggest
moving it to a branch, e.g., 'svnmover'? If someone wishes to continue the
experiment in the future, the branch could be a starting point, or at least
exist as a reference of what has been tried.
Let me know if you like this idea. If so, then after you commit the patch,
I'll be happy to make the branch, reverse-cherrypick it there, and write
the BRANCH-README.
If you don't think it's worth saving, even in a branch, that's okay too.
Cheers,
Nathan
