Philip Martin wrote:
> The problem is that ra_dav is the only layer that uses wcprops, if you
> only test over ra_local you won't see wcprops related failures.
>
> The original problem generated some dev list discussion between me and
> Karl, and some code from Karl. As I recall the problem was that an
> interrupted update caused the wcprops and the text-base/revision to be
> out-of-sync (I can't remember which one was wrong). I don't know
> whether the wcprops code is still fragile, or whether Karl's change
> fixed it.
Ok, I did some testing over ra_dav and found no problems. It passes
the whole test-suite, plus I did quite a bit of checkouts/updating and
interrupting without any issues. I also looked at issue #1136, which
I think is what you were referring to here. I was unable to reproduce
that failure with the new code either.
One last concern. Do we need to be worried about compatability? This
change would prevent trunk from running the log files from older
working copies since their names have changed. Should I hack in
something to try "log" before trying "log.1" or can we just accept
that amount of change?
-Josh
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Sun May 23 21:48:00 2004