Re: Ruby makefile maker (not really [PATCH] INSTALL...)

Sean Russell <ser@germane-software.com> writes:

> Attached to this email is an extconf.rb file for the Ruby binding. This
> should go into the subversion/bindings/ruby directory. This is the standard
> way to build makefiles for native libraries for Ruby, and it uses a toolkit
> distributed with the standard Ruby distribution. The user runs this file
> (with ruby) and the toolkit creates a Makefile for the target (in this case
> libsvnruby) based on the user's system configuration.

Thanks. I have a few comments if you want to refine
extonf.rb more. It would be great if you could address
those and make extconf.rb more generalized. Then it could
be added to the repository.

> 0) There was no makefile in this directory; hence my submitting this

The main reason is that I haven't had time to improve my
extconf.rb. Another reason is that since Subversion has
only one central Makefile, I wanted to compile Ruby binding
from that Makefile. However, it would be nice for some user
to be able to just check out Ruby bindings and build it
without reconfiguring whole Subversion again, so I'm not
against adding extconf.rb to the repository.

> 1) I'm assuming that libsvnruby.so is the name of the target library

Actually, svn.so is the name of the library. Other Ruby
DSOs are named similar way like zlib.so and digest.so.

> 2) I haven't tested this on Windows, or anything other than Linux, for that
> matter.

So do I. :-)

> 3) It assumes that svn is already installed; it looks for the svn libraries,
> in particular, to be installed in a global library path.

Perhaps it needs dir_config(svn) so that one can specify
--with-svn-dir=/opt/svn or something.

Sean Russell <ser@germane-software.com> writes:

> Second, would someone check to make sure that all of the libraries I'm
> requiring are actually required? They're listed on line 23, and are libX,
> where is in:
>
> apr dl mm crypt pthread svn_subr neon svn_delta expat svn_client svn_wc svn_ra

There are libsvn_fs and libsvn_repos missing. And apr stuff
should come from apr-config --libs. Also, CFLAGS should
come from apr-config --cflags instead of hardcoding -pthread and -O.

-- 
Yoshiki Hayashi
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org