[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: Julian Foad <julianfoad_at_btopenworld.com>
Date: Fri, 29 Feb 2008 11:30:59 +0000

David Glasser wrote:
> On Thu, Feb 28, 2008 at 12:40 PM, Julian Foad wrote:
>> It's not that some of these commands are susceptible to race conditions and
>> others are not: they all are. It's just a question of degree and likelihood.
>>
>> Example:
>> [[[
>> INITIALLY
>> svn:ignore = *.tmp
>>
>> USER1
>> # Temporary files don't need to be ignored any more:
>> svn propdel svn:ignore
>>
>> USER2
>> # Adds "*.bak":
>> svn propset svn:ignore "*.tmp
>> *.bak"
>>
>> RESULT
>> Same race condition as if USER1 had been using "propset".
>> ]]]
>
> Well, actually, yeah: it's the same race condition, and thus would be
> prevented by the same requirement to use baserev for propset :-)

No, I meant USER1 and USER2 are acting asynchronously. If USER1 gets there
first, then USER2's enforced use of a base revision makes him stop and
re-think. If USER2 gets there first, USER1 silently overwrites USER2's change
if he doesn't specify a base revision. They're symmetrical.

- Julian

---------------------------------------------------------------------
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-29 12:31:31 CET

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.