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

Re: svn propedit URL

From: Garrett Rooney <rooneg_at_electricjellyfish.net>
Date: 2006-09-29 17:25:45 CEST

On 9/29/06, David Glasser <glasser@mit.edu> wrote:
> (I assume there must have been discussion of this before, but I'm
> having trouble finding it.)
> It would be useful to support "svn propedit" on URLs. The obvious
> problem with modifying properties on URLs is accidentally overwriting
> somebody else's changes. While this is clearly a problem for propset
> and propdel, it's not as big of a deal (in theory) for propedit: svn
> could remember the revision of the property being edited, and pass it
> through the various layers of propset, and the operation would fail if
> it was out of date. This can be implemented with the current RA API;
> it just needs a new client API (and 'svn' changes).
> I'm specifically proposing the following API change. What do folks
> think?

There are two gotchas with edit type commands on URLs. Neither are
showstoppers, but you need to be aware of them. First, you need to be
sure to avoid race conditions where you start editing, someone commits
a change, and then you hit save and commit. With files this can be
handled by remembering what version of the file you started with and
using that as your initial rev during the commit. There should be a
similar thing you can do with properties. Second, finding a UI for
entry of log messages, since it's kind of lame to pop up one editor to
make your change, then a second one to enter a log. I don't
personally have an issue with that being the UI, but other people have
expressed problems with it in the past.


To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Fri Sep 29 17:26:08 2006

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.