julianf...@apache.org writes:

> Author: julianfoad
> Date: Tue Apr  2 19:53:43 2013
> New Revision: 1463721
> after releasing it in other places.  We should pick one and be consistent.

> --- subversion/trunk/subversion/libsvn_client/commit.c (original)
> +++ subversion/trunk/subversion/libsvn_client/commit.c Tue Apr  2 19:53:43 
> 2013
> @@ -985,10 +985,11 @@ svn_client_commit6(const apr_array_heade
>                     iterpool);
>      }
>  
> + cleanup:
>    /* Sleep to ensure timestamp integrity. */
> -  svn_io_sleep_for_timestamps(base_abspath, pool);
> +  if (commit_in_progress)
> +    svn_io_sleep_for_timestamps(base_abspath, pool);

This bit doesn't look right.  commit_in_progress is set back to FALSE
before post-commit processing so as far as I can see a successful commit
no longer sleeps.  This ould explain the buildbot failures
http://ci.apache.org/builders/svn-x64-centos-gcc/builds/9039

-- 
Certified & Supported Apache Subversion Downloads:
http://www.wandisco.com/subversion/download

Reply via email to