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

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
Received on 2014-05-19 18:56:46 CEST