On Thu, Mar 22, 2012 at 4:41 AM, Philip Martin
<philip.martin_at_wandisco.com> wrote:
> There is another failure in the ruby testsuite:
>
> http://ci.apache.org/builders/svn-x64-ubuntu-gcc/builds/4626
>
> 1) Failure:
> test_changelists_get_with_block(SvnClientTest)
> /var/lib/buildbot/svn-buildslave/svn-x64-ubuntu/build/subversion/bindings/swig/ruby/test/test_client.rb:2296:in `assert_changelists'
> /var/lib/buildbot/svn-buildslave/svn-x64-ubuntu/build/subversion/bindings/swig/ruby/test/util.rb:204:in `make_context'
> /var/lib/buildbot/svn-buildslave/svn-x64-ubuntu/build/subversion/bindings/swig/ruby/test/test_client.rb:2288:in `assert_changelists'
> /var/lib/buildbot/svn-buildslave/svn-x64-ubuntu/build/subversion/bindings/swig/ruby/test/test_client.rb:2349:in `test_changelists_get_with_block':
> <{nil=>
> ["/tmp/d20120322-8616-qtl2ah/wc",
> "/tmp/d20120322-8616-qtl2ah/wc/hello1.txt",
> "/tmp/d20120322-8616-qtl2ah/wc/hello2.txt"]}> expected but was
> <{nil=>
> ["/tmp/d20120322-8616-qtl2ah/wc",
> "/tmp/d20120322-8616-qtl2ah/wc/hello2.txt",
> "/tmp/d20120322-8616-qtl2ah/wc/hello1.txt"]}>.
>
> The failing code is:
>
> make_context(log) do |ctx|
> File.open(path1, "w") {|f| f.print(src)}
> File.open(path2, "w") {|f| f.print(src)}
> ctx.add(path1)
> ctx.add(path2)
> ctx.commit(@wc_path)
>
> assert_equal({}, yield(ctx, changelist1))
> assert_equal({nil=>[@wc_path,path1,path2].map{|f| File.expand_path(f)}}, yield(ctx, nil))
>
> This is likely to be APR's new hash randomisation which means that the
> notification order of "hello1.txt" and "hello2.txt" is unpredictable. I
> think the fix is for the lists to be sorted but I don't know the
> necessary Ruby.
>
> --
> uberSVN: Apache Subversion Made Easy
> http://www.uberSVN.com
I'm sure you are right. I'll start into this on the weekend....
--
Joe
Received on 2012-03-22 15:36:48 CET