[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

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

From: Julian Foad <julianfoad_at_btopenworld.com>
Date: Mon, 19 May 2014 17:53:19 +0100 (BST)

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

This is an archived mail posted to the Subversion Dev mailing list.