On Fri, Nov 07, 2003 at 05:52:59PM -0600, C. Michael Pilato wrote:
> Brian Denny <brian@briandenny.net> writes:
>
> > If I understand issue #929 correctly (the issue that the svn_io_temp_dir
> > function was intended to fix), the problem was that read-only operations
> > were failing if the user didn't have write permissions in the working
> > directory...
> >
> > And yep, the patch below (representing the naive solution) breaks those
> > certain diff operations when run in a directory without write permissions.
> >
> >
> > Suggestions?
>
> Perhaps you could enumerate the set of operations, and then point out
> which ones you expect to work and not work with today's code, and
> after your patch?
Here's what I know:
Say we start out with (r3):
wc/
wc/B/
wc/C/
wc/C/F/
Then we add and commit (r4)
wc/C/F/foo
wc/C/F/Q/
wc/C/F/Q/bar
Now the following merge command fails with HEAD, but works with my patch:
wc$ svn merge -r3:4 $REPO/C/F B
The following works with HEAD, but fails with my patch:
wc$ chmod -R a-w .
wc$ svn diff $REPO/C/F $REPO/B
Are there other operations that might fail in HEAD, or succeed in HEAD
but fail with my patch? Maybe. Perhaps others who know the code better
could say, but to me based on an hour or so of study, I have no idea.
-brian
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Sat Nov 8 04:17:30 2003