Re: svn commit: r921445 - in /subversion/trunk/subversion: include/private/svn_wc_private.h include/svn_wc.h libsvn_client/commit.c libsvn_wc/entries.c

[Joe Swatosh]

Hi Philip

On Wed, Mar 10, 2010 at 8:47 AM,  <phi...@apache.org> wrote:
> Author: philip
> Date: Wed Mar 10 16:47:27 2010
> New Revision: 921445
>
> URL: http://svn.apache.org/viewvc?rev=921445&view=rev
> Log:
> Remove some access batons from post-commit processing.
>


>

Since this patch the Ruby bindings test has been failing on the Ubuntu
build bot (and locally for me).  I haven't looked at the patch closely
so I'm not sure what's up, but the relevant part of the test is:

      Svn::Wc::AdmAccess.open(nil, @wc_path, true, 5) do |access|
        assert_raises(Svn::Error::WcPathFound) do
          access.mark_missing_deleted(path1)
        end
        FileUtils.rm(path1)
        access.mark_missing_deleted(path1)  ### This line raises
Svn::Wc::WcPathFound ###
        access.maybe_set_repos_root(path2, @repos_uri)
      end

Judging by the name of the function, it appears to me that the test is
correct....
I double checked, the rm is removing the file before the call.  Could
you check this please, or suggest how the test should be modified?

Thanks,

Joe