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

Re: svn commit: r25517 - in trunk/subversion: include libsvn_client

From: Karl Fogel <kfogel_at_red-bean.com>
Date: 2007-07-06 18:12:39 CEST

"David Glasser" <glasser@mit.edu> writes:
>> The reason why I added a second macro is to make it clear to users of
>> the SVN_DEPTH_FROM_RECURSE macro that it doesn't give the correct
>> results in all cases, more specifically when working with status.
>
> In fact it ought to be used in more places, like
> svnserve/serve.c(status)... (and I think somewhere in mod_dav_svn as
> well)

I believe the status command is special and doesn't do the same
recurse->depth mappings as other commands, and that's why it doesn't
use the macro.

Checking... yup: it maps nonrecursive to svn_depth_empty instead of
svn_depth_files.

-Karl

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Fri Jul 6 19:43:38 2007

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