Fumitoshi UKAI wrote:
>At Thu, 26 Dec 2002 19:03:53 -0500,
>Garrett Rooney wrote:
>
>
>
>>>I check it can be loaded without errors, but not yet tried much
>>>operations.
>>>Is there any good sample/test program or documentation?
>>>
>>>
>>unfortunately, no. this stuff could really use a good test suite and a
>>heavy dose of documentation, but there currently isn't any. again, if
>>you write it, i'll commit it ;-)
>>
>>
>
>Ok, I'll write it.
>
>
i look forward to seeing them.
>
>
>
> * subversion/bindings/ruby/client.c
> (cl_log_message_func): add tmp_file arugments because it is
> svn_client_get_commit_log_t.
> It doesn't use tmp_file, so it sets NULL.
> (cl_propget): revision and auth_baton are required by
> svn_client_propget().
> (cl_proplist): revision and auth_baton are required by
> svn_client_proplist().
>
> * subversion/bindings/ruby/extconf.rb
> add comments how to build in build tree.
> hack to use libtool in case ruby binding built in build tree.
> Some local libs (-lgdbm, -lexpat, -ldb-4) are required,
> but missing from apr-config. Without this hack, have_library() failed.
> Add build tree to LDFLAGS to link library currently built.
> Add subversion/include to include path.
> library name fixes: s/apr-0/apr/, s/aprutil-0/aprutil/.
> Except classical apr path layout, no -0 suffix is used.
>
this all looks fine from what i can see. i'll take a closer look
tonight, when i'm in front of a machine, and i'll commit everything
except the s/apr-0/apr/ s/aprutil-0/aprutil/ part. we build against the
stock apr/aprutil (at least theoretically). if debian wants to change
the names of the libraries, they'll have to patch this locally.
thanks a lot,
-garrett
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Tue Jan 7 19:56:49 2003