You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@subversion.apache.org by Ben Collins-Sussman <su...@newton.ch.collab.net> on 2001/02/17 15:46:10 UTC

Re: CVS update: subversion/subversion/libsvn_client ra_loader.c

gstein@tigris.org writes:

>   User: gstein  
>   Date: 01/02/17 03:58:44
> 
>   Modified:    subversion/libsvn_client ra_loader.c
>   Log:
>   Enable DSO loading and "prelinked" support for RA libraries.

Greg, you rock.  :)


>   +    /* ### fix the .so part */
>   +    libname = apr_psprintf (pool, "libsvn_ra_%s.so", ra_name);

Yeah, this was bugging me too.  Doesn't APR define various .so
suffixes portably?

Re: CVS update: subversion/subversion/libsvn_client ra_loader.c

Posted by Greg Stein <gs...@lyra.org>.

On Sat, Feb 17, 2001 at 09:46:10AM -0600, Ben Collins-Sussman wrote:
> gstein@tigris.org writes:
>...
> >   +    /* ### fix the .so part */
> >   +    libname = apr_psprintf (pool, "libsvn_ra_%s.so", ra_name);
> 
> Yeah, this was bugging me too.  Doesn't APR define various .so
> suffixes portably?

Not yet.

libtool won't really tell us, either. I'll be adding stuff to APR, and
eventually fix the above code. Until then, the .so suffix is fine.

Cheers,
-g

-- 
Greg Stein, http://www.lyra.org/