svn commit: r1586255 - in /subversion/trunk/subversion: libsvn_client/prop_commands.c tests/cmdline/prop_tests.py

[Bert Huijben]

-----Original Message-----
Sent: donderdag 10 april 2014 12:14
libsvn_client/prop_commands.c tests/cmdline/prop_tests.py
Author: rhuijben
Date: Thu Apr 10 10:14:23 2014
New Revision: 1586255
URL: http://svn.apache.org/r1586255
Replace assertions triggered in svn_dirent_get_absolute() triggered by
$ svn pl -r PREV https://svn.apache.org/repos/asf
$ svn pg Q -r PREV https://svn.apache.org/repos/asf
and
$ tortoiseproc /command:properties /path:https://svn.apache.org/repos/asf
(which tries to obtain the WORKING properties)
with a proper error.

Hi,

This last error was found by CollabNet QA and was the reason I applied this patch.

It would be nice if this command could default to using HEAD instead of WORKING when '/path'' specifies a URL, just like 'svn'

We will probably fix some of our code by explicitly adding /rev:HEAD to the commandline, to avoid the problem on both old and new versions of TortoiseSVN.

Bert

------------------------------------------------------
http://tortoisesvn.tigris.org/ds/viewMessage.do?dsForumId=757&dsMessageId=3076274

To unsubscribe from this discussion, e-mail: [dev-***@tortoisesvn.tigris.org].

[Stefan Küng]

Post by Bert Huijben

-----Original Message-----
Sent: donderdag 10 april 2014 12:14
libsvn_client/prop_commands.c tests/cmdline/prop_tests.py
Author: rhuijben
Date: Thu Apr 10 10:14:23 2014
New Revision: 1586255
URL: http://svn.apache.org/r1586255
Replace assertions triggered in svn_dirent_get_absolute() triggered by
$ svn pl -r PREV https://svn.apache.org/repos/asf
$ svn pg Q -r PREV https://svn.apache.org/repos/asf
and
$ tortoiseproc /command:properties /path:https://svn.apache.org/repos/asf
(which tries to obtain the WORKING properties)
with a proper error.

Hi,
This last error was found by CollabNet QA and was the reason I applied this patch.
It would be nice if this command could default to using HEAD instead of WORKING when '/path'' specifies a URL, just like 'svn'
We will probably fix some of our code by explicitly adding /rev:HEAD to the commandline, to avoid the problem on both old and new versions of TortoiseSVN.

Fixed in r25414.

Stefan
--
___
oo // \\ "De Chelonian Mobile"
(_,\/ \_/ \ TortoiseSVN
\ \_/_\_/> The coolest interface to (Sub)version control
/_/ \_\ http://tortoisesvn.net

------------------------------------------------------
http://tortoisesvn.tigris.org/ds/viewMessage.do?dsForumId=757&dsMessageId=3076304

To unsubscribe from this discussion, e-mail: [dev-***@tortoisesvn.tigris.org].