subversion-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Julian Foad <julianf...@btopenworld.com>
Subject Re: svn commit: r1592724 - in /subversion/trunk/subversion: include/svn_client.h svn/propget-cmd.c tests/cmdline/prop_tests.py tests/cmdline/svnadmin_tests.py
Date Mon, 19 May 2014 16:53:19 GMT
Branko Čibej wrote:
> On 19.05.2014 13:30, Daniel Shahaf wrote:
>> Because I assumed it would error, realized it didn't, and digging into
>> the history revealed that it was probably an unintentional omission. 
[...]
>> I hadn't checked 'svn propget versioned_prop'.  It makes sense to me that it
>> should error out on non-existing properties. Would that be a reasonable course
>> to take?  Or does anyone prefer to retain the pre-r1592724 behaviour?
> 
> Didn't we spend time making sure that recursive operations do /not/
> error out in such cases?

That sounds possible.

Another thought: what is the impact on the XML output mode? That's a more important API, and
I haven't reviewed the impact at all.

- Julian

Mime
View raw message