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

Reply via email to