Re: Pristines-on-demand: authz denied during textbase sync (#4888)

2022-04-17 Thread Daniel Shahaf
Julian Foad wrote on Wed, Apr 06, 2022 at 13:07:43 +0100: > > Filed as issue #4888, https://subversion.apache.org/issue/4888 > > I have just been looking back over this issue. Clearly there is more to > it than a quick fix. Summary, based on reviewing the email thread: > > - FAIL: authz_tests.py

Re: Pristines-on-demand: authz denied during textbase sync (#4888)

2022-04-06 Thread Karl Fogel
On 06 Apr 2022, Julian Foad wrote: Evgeny Kotkov wrote: - revert the patch I applied, as it's papering over the problem in an incomplete way and so possibly causes more confusion than it fixes. - leave this issue open and come back to it later; it's an edge case not part of common work flow

Re: Pristines-on-demand: authz denied during textbase sync (#4888)

2022-04-06 Thread Julian Foad
Evgeny Kotkov wrote: >> - revert the patch I applied, as it's papering over the problem in an >> incomplete way and so possibly causes more confusion than it fixes. >> >> - leave this issue open and come back to it later; it's an edge case not >> part of common work flows. > > +1. Thanks! Revert

Re: Pristines-on-demand: authz denied during textbase sync (#4888)

2022-04-06 Thread Evgeny Kotkov
Julian Foad writes: > I suggest: > > - revert the patch I applied, as it's papering over the problem in an > incomplete way and so possibly causes more confusion than it fixes. > > - leave this issue open and come back to it later; it's an edge case not > part of common work flows. +1. Regards

Re: Pristines-on-demand: authz denied during textbase sync (#4888)

2022-04-06 Thread Julian Foad
> Filed as issue #4888, https://subversion.apache.org/issue/4888 I have just been looking back over this issue. Clearly there is more to it than a quick fix. Summary, based on reviewing the email thread: - FAIL: authz_tests.py 31 remove_access_after_commit - Patched in

Re: Pristines-on-demand: authz denied during textbase sync (#4888)

2022-02-25 Thread Julian Foad
Filed as issue #4888, https://subversion.apache.org/issue/4888 I will come back to it soonish.