On Thu, Apr 11, 2013 at 12:28 AM, <julianf...@apache.org> wrote: > Author: julianfoad > Date: Wed Apr 10 20:28:42 2013 > New Revision: 1466659 > > URL: http://svn.apache.org/r1466659 > Log: > Fix issue #4348, "missing directory replace on update 1.8 svnserve 1.7 > client". > > * subversion/svnserve/serve.c > (update): Correct the backward-compatibility code for the > 'ignore_ancestry' parameter introduced in r1465292. > > Found by: philip > > Modified: > subversion/trunk/subversion/svnserve/serve.c > > Modified: subversion/trunk/subversion/svnserve/serve.c > URL: > http://svn.apache.org/viewvc/subversion/trunk/subversion/svnserve/serve.c?rev=1466659&r1=1466658&r2=1466659&view=diff > ============================================================================== > --- subversion/trunk/subversion/svnserve/serve.c (original) > +++ subversion/trunk/subversion/svnserve/serve.c Wed Apr 10 20:28:42 2013 > @@ -1814,7 +1814,7 @@ static svn_error_t *update(svn_ra_svn_co > const char *target, *full_path, *depth_word; > svn_boolean_t recurse; > apr_uint64_t send_copyfrom_args; /* Optional; default FALSE */ > - apr_uint64_t ignore_ancestry; /* Optional; default TRUE */ > + apr_uint64_t ignore_ancestry; /* Optional; default FALSE */ > /* Default to unknown. Old clients won't send depth, but we'll > handle that by converting recurse if necessary. */ > svn_depth_t depth = svn_depth_unknown; > @@ -1842,7 +1842,7 @@ static svn_error_t *update(svn_ra_svn_co > conn, pool, b, rev, target, NULL, TRUE, > depth, > (send_copyfrom_args == TRUE) /* send_copyfrom_args > */, > - (ignore_ancestry != FALSE) /* ignore_ancestry */)); > + (ignore_ancestry == TRUE) /* ignore_ancestry */)); As far I remember any non-zero value considered as boolean 'true'. So correct expression should be just 'ignore_ancestry'.
-- Ivan Zhakov