subversion-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Daniel Shahaf <...@daniel.shahaf.name>
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 Wed, 21 May 2014 11:26:13 GMT
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

Mime
View raw message