[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: [PATCH] svnadmin build-repcache command

From: Daniel Shahaf <d.s_at_daniel.shahaf.name>
Date: Wed, 25 Mar 2020 21:37:49 +0000

> > I'm proposing to modify fs_has_rep_sharing() to return True when running
> > on FSFS with rep-sharing enabled and to return False on other backends.
> > That's separate from your test's decorators.
> >
>
> I think that this might be unrelated to this patch, but also I may be
> misunderstanding
> what exactly you propose. If we modify fs_has_rep_sharing() to return True
> for FSFS and to return False for other backends, we will have a problem
> with FSX,
> which also supports rep-sharing.

The function, as it stands, assumes that the tests run on FSFS, and
will return wrong values when that assumption doesn't hold. The point
is to make that function return the correct value in every configuration.
If the fix I proposed is wrong under FSX, then the issue should be fixed
differently.
Received on 2020-03-25 22:38:02 CET

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.