Re: [PATCH] enabling ruby in the subversion build

2012-11-07 Thread Michael Chletsos
Hi Philip - Not sure if you remember me from the Subversion Live! event the other week (Michael with Assembla), anyhow . . . I was able to run the ruby swig tests - though one test failed because of encoding, not a huge deal breaker, and the regular make test ran fine - were you talking about some

Re: [PATCH] enabling ruby in the subversion build

2012-11-07 Thread Michael Chletsos
etty poor, create /usr/bin/ruby: #!/bin/sh export RUBY_HEAP_MIN_SLOTS=60 export RUBY_GC_MALLOC_LIMIT=5900 export RUBY_FREE_MIN=20 exec "/usr/bin/ruby-1.9.3-p286" "$@" then test: ruby -ropenssl -rzlib -rreadline -ryaml -e "puts :success" This should get r

Re: [PATCH] enabling ruby in the subversion build

2012-11-07 Thread Michael Chletsos
I would support 1.9.3 - ruby 1.9.2 and 1.9.1 are not considered good versions and are problematic. The current patch I used does not fix the configure script, I could modify that to only use <= 1.8.7 or >= 1.9.3 if you like. On Wed, Nov 7, 2012 at 5:31 AM, Philip Martin wrote: >

Re: [PATCH] enabling ruby in the subversion build

2012-11-07 Thread Michael Chletsos
Sorry for delay. Patch for configure is attached. It also updates Config to RbConfig which will remove annoying errors and is backwards compatible. On Wed, Nov 7, 2012 at 5:43 AM, Philip Martin wrote: > Michael Chletsos writes: > >> The current patch I used does not fix the conf

Re: [PATCH] enabling ruby in the subversion build

2012-11-07 Thread Michael Chletsos
sorry about confusion - here is the updated patch, with swig updates. On Wed, Nov 7, 2012 at 8:06 AM, Philip Martin wrote: > Michael Chletsos writes: > >> Sorry for delay. Patch for configure is attached. > > You don't patch configure, that's a generated fil

Re: [PATCH] enabling ruby in the subversion build

2012-11-07 Thread Michael Chletsos
Sorry - I am having one of those morning - this one should be good, I keep reversing my files and my thoughts . . . I will check out trunk later and fix the patches and this patch and make them all nice and be real. On Wed, Nov 7, 2012 at 8:54 AM, Peter Samuelson wrote: > > [Michael Ch

Re: [PATCH] enabling ruby in the subversion build

2012-11-07 Thread Michael Chletsos
Here is a complete patch that I tested - more work needs to be done on the tests, but currently all tests that are run, succeed on my system. I based this on trunk today. On Wed, Nov 7, 2012 at 8:57 AM, Michael Chletsos wrote: > Sorry - I am having one of those morning - this one should be g

Re: [PATCH] enabling ruby in the subversion build

2012-11-07 Thread Michael Chletsos
OK - this should fix your issues, tested on ruby1.9.3 and ruby1.8.7 - passed for me, let me know what you get. On Wed, Nov 7, 2012 at 4:40 PM, Philip Martin wrote: > Michael Chletsos writes: > >> Here is a complete patch that I tested - more work needs to be done on >> the te

Re: [PATCH] enabling ruby in the subversion build

2012-11-08 Thread Michael Chletsos
ruby1.8 * configure.ac: detect ruby1.9.3 - added teeny version detection restrict build to 1.8.x and >= 1.9.3 Patch by: Michael Chletsos , Vincent Batts On Thu, Nov 8, 2012 at 1:03 AM, Philip Martin wrote: > Michael Chletsos writes: > >> OK - this should fix your

Re: [PATCH] enabling ruby in the subversion build

2012-11-08 Thread Michael Chletsos
Thank you! Ruby community will be excited, I will continue to improve the tests. On Thu, Nov 8, 2012 at 8:29 AM, Philip Martin wrote: > Michael Chletsos writes: > >> Initial support for ruby 1.9.3 when creating swig bindings for subversion. > > I tweaked the format a littl