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

Re: [PATCH] Add URL support for propset and propdel and fix URL support for propedit

From: Mark Phippard <markphip_at_gmail.com>
Date: Fri, 22 Feb 2008 15:09:14 -0500

On Fri, Feb 22, 2008 at 3:04 PM, Karl Fogel <kfogel_at_red-bean.com> wrote:
> kmradke_at_rockwellcollins.com writes:
> > Needed a way to specify a "base revision", because of the possible
> > race condition between looking at the properties and then
> > performing a command directly on the repository.
> >
> > We kicked around a --base-rev parameter in another thread, but
> > it was easier for me to "overload" the meaning of -r here.
> >
> > Not using -r and using some other parameter is definitely a
> > (probably less confusing) option.
> >
> > We could also assume "HEAD" (others didn't like this)
> >
> > Is peg revision syntax appropriate? (no idea on this)
>
> FWIW, I like the "-rACTUAL_NUMBER" solution a lot. It protects
> against any possibility of race condition.
>
> It's okay if HEAD is different from ACTUAL_NUMBER, as long as the
> property still has the value in HEAD that it did in ACTUAL_NUMBER,
> of course.

Yeah, I just did not grok that in the original message.

-- 
Thanks
Mark Phippard
http://markphip.blogspot.com/
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe_at_subversion.tigris.org
For additional commands, e-mail: dev-help_at_subversion.tigris.org
Received on 2008-02-22 21:09:25 CET

This is an archived mail posted to the Subversion Dev mailing list.