[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: Daniel Shahaf <d.s_at_daniel.shahaf.name>
Date: Wed, 21 May 2014 11:26:13 +0000

I've filed http://subversion.tigris.org/issues/show_bug.cgi?id=4505 to
track this.

Julian Foad wrote on Mon, May 19, 2014 at 17:53:19 +0100:
> 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-21 13:27:08 CEST

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