Re: [PATCH] Improve error checking for rev-prop commands (incl. issue #2364)

[Julian Foad <ju...@btopenworld.com>]

Ben Gollmer wrote:
> Julian Foad wrote:
> 
>>Issue #2364 says "--revprop doesn't work with -r BASE/COMMITTED/PREV but
>>[does] with HEAD".
>>
>>My analysis is that there are two issues here.
>>
>>1. It was never intended that a working copy item should be specified as
>>the target for operating on a revision property.  It is accepted, but
>>the documentation doesn't mention it: the help says "[URL]".  Therefore
>>there is no reason why a revision should be able to be specified
>>relative to a WC target.
> 
> This won't block custom revprops from being set (issue 1976), will it? I

No.  All this is saying is that you can't specify the revision by reference to 
a working-copy path with "BASE" or "COMMITTED" or "PREV"; you can still specify 
a working copy path and "HEAD", or you can specify (a WC path or URL) with (a 
revision number or date).  Plenty of options!

> guess as long as the `svn propset --revprop` syntax isn't used to set
> them, it won't be an issue.

Er, well, indeed, it won't be an issue if you have some other way of setting 
them ... but then this won't stop you from setting any rev-props you like even 
if you do use the normal "svn propset --revprop" syntax.

- Julian

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org