Re: APR hash order ruby test failure

2012-04-06 Thread Joe Swatosh
On Fri, Apr 6, 2012 at 11:32 AM, Joe Swatosh wrote: > On Wed, Mar 28, 2012 at 2:30 AM, Philip Martin > wrote: >> Philip Martin writes: >> >>> There is another failure in the ruby testsuite: >>> >>> http://ci.apache.org/builders/svn-x64-ubuntu-gcc/builds/4626 >>> >>>   1) Failure: >>> test_change

Re: APR hash order ruby test failure

2012-04-06 Thread Joe Swatosh
On Wed, Mar 28, 2012 at 2:30 AM, Philip Martin wrote: > Philip Martin writes: > >> 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-buil

Re: APR hash order ruby test failure

2012-03-29 Thread Philip Martin
Joe Swatosh writes: > From that perspective, it seems to me that the best approach is to use > the Set in the test, but not in the bindings themselves. Also the > current APIs don't provide order guarantees (obviously), but if they > started to guarantee order in the future, the Set may be a bad

Re: APR hash order ruby test failure

2012-03-28 Thread Joe Swatosh
On Wed, Mar 28, 2012 at 5:44 AM, Philip Martin wrote: > Hyrum K Wright writes: > >> On Wed, Mar 28, 2012 at 4:30 AM, Philip Martin >>> >>> We fix these by converting the arrays to sets so that the order is >>> irrelevant.  The open question is whether we do this solely within the >>> testsuite, c

Re: APR hash order ruby test failure

2012-03-28 Thread Hyrum K Wright
On Wed, Mar 28, 2012 at 8:04 AM, Daniel Shahaf wrote: > Philip Martin wrote on Wed, Mar 28, 2012 at 10:30:31 +0100: >> /var/lib/buildbot/svn-buildslave/svn-x64-ubuntu/build/subversion/bindings/swig/ruby/test/test_wc.rb:764:in >> `test_diff_callbacks_for_backward_compatibility': >> <[[:dir_props_c

Re: APR hash order ruby test failure

2012-03-28 Thread Daniel Shahaf
Philip Martin wrote on Wed, Mar 28, 2012 at 10:30:31 +0100: > /var/lib/buildbot/svn-buildslave/svn-x64-ubuntu/build/subversion/bindings/swig/ruby/test/test_wc.rb:764:in > `test_diff_callbacks_for_backward_compatibility': > <[[:dir_props_changed, > "/tmp/d20120327-16259-17t4nz8/wc", > [# @n

Re: APR hash order ruby test failure

2012-03-28 Thread Philip Martin
Hyrum K Wright writes: > On Wed, Mar 28, 2012 at 4:30 AM, Philip Martin >> >> We fix these by converting the arrays to sets so that the order is >> irrelevant.  The open question is whether we do this solely within the >> testsuite, changing both sides of the comparison, or whether we change >> t

Re: APR hash order ruby test failure

2012-03-28 Thread Hyrum K Wright
On Wed, Mar 28, 2012 at 4:30 AM, Philip Martin wrote: > Philip Martin writes: > >> 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-buil

Re: APR hash order ruby test failure

2012-03-28 Thread Philip Martin
Philip Martin writes: > 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/te

Re: APR hash order ruby test failure

2012-03-22 Thread Daniel Shahaf
Joe Swatosh wrote on Thu, Mar 22, 2012 at 07:36:16 -0700: > On Thu, Mar 22, 2012 at 4:41 AM, Philip Martin > 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(SvnC

Re: APR hash order ruby test failure

2012-03-22 Thread Joe Swatosh
On Thu, Mar 22, 2012 at 4:41 AM, Philip Martin 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/subvers

APR hash order ruby test failure

2012-03-22 Thread Philip Martin
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_change