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

Re: svn commit: r30161 - in trunk/subversion: libsvn_ra_neon tests/cmdline

From: Karl Fogel <kfogel_at_red-bean.com>
Date: Fri, 04 Apr 2008 17:41:05 -0400

"David Glasser" <glasser_at_davidglasser.net> writes:
> I was hoping that somebody with a better understanding of our DAV
> report protocol could see if there are any other similar changes to be
> made as well first.

I think this change is spiffy (just reviewed), and you underestimate
your understanding of the DAV protocol :-).

You're right that we could open up "add-directory" to tolerate
"delete-entry" too, but I'm wary of making a change that no bug report
has called for yet. As you say, we don't send add-dir-with-history yet,
and it seems to me that when we do, this either will or won't (but
probably "will") be a problem, at which point we'll notice it. My
instinct is to leave the rest alone right now, but if we do change it,
let's keep the change in trunk and at least not backport it to 1.5
(which, in theory, would never need it).

By the way, I looked in ra_serf, and it doesn't seem to have any of
these problems. start_report() just handles whatever it sees.

To unsubscribe, e-mail: dev-unsubscribe_at_subversion.tigris.org
For additional commands, e-mail: dev-help_at_subversion.tigris.org
Received on 2008-04-04 23:41:19 CEST

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.