Re: tortoiseproc /command:revprop /path:"repo-url" /rev:1234

Stefan Küng wrote:

>On 29.03.2012 09:27, Lorenz wrote:
>> Lorenz wrote:
>>> Stefan Küng wrote:
>>>> On 27.03.2012 16:00, Lorenz wrote:
>>>>> would you consider adding a new command to tortoiseproc that opens the
>>>>> rev-prop editor dialog?
>>>>> [..]
>>>>
>>>> Done in r22733:
>>>> /command:properties /path:http://something /rev:1234
>>
>> just grabbed the nightly (TortoiseSVN 1.7.99, Build 22736 - 64 Bit
>> -dev, 2012/03/28 21:57:05) and the new command works fine so far as it
>> brings up the revision properties dialog.
>>
>> Could we go a step further and optionally extend the path parameter
>> with a property name? In which case the properties editor could be
>> opened directly for that property?
>>
>> Perhaps '/path:url*property-name' would do.
>
>Done in r22743 as:
>/property:name
>instead of misusing the /path parameter for this.

thanks again

>But it still opens the main properties dialog first and then opens the
>dialog for the specified property.

no problem, a single ESC takes care of that 8-)

>> Some additional thoughts:
>>
>> 1)
>> Is seems, that the properties command did only work on working copies
>> in the past?
>> I would not have expected that as the repo browser allows to review,
>> and for the head revision even to edit, the versioned properties too.
>>
>> Don't get me wrong, I don't need this functionality, but should it be
>> implemented in the future, a separate command or additional parameter
>> would be needed to differentiate between 'view/edit revprops' and
>> 'view versioned props' at the given revision.
>
>You're right, but right now I don't think this feature is currently
>needed for automation.

ok with me. But then this restriction should be mentioned in command
description (see attached patch for my proposal) because ...

>> 2)
>> There is no error message if the path is wrong (not pointing to a wc
>> or existing folder in a repo).
>>
>> That too is not relevant in my case, because when calling the command
>> from my script I have a valid url.
>
>TSVN does rarely check the automation parameters. That's up to the caller.

        tortoiseproc /command:properties /path:"valid-URL"

opens the properties dialog, suggesting that it's working on versioned
properties directly in the repo too.

-- 
Lorenz
------------------------------------------------------
http://tortoisesvn.tigris.org/ds/viewMessage.do?dsForumId=4061&dsMessageId=2947950
To unsubscribe from this discussion, e-mail: [users-unsubscribe_at_tortoisesvn.tigris.org].