Re: svn commit: rev 7861 - in trunk: . subversion/bindings/swig/perl

On Sat, Nov 29, 2003 at 01:03:24PM -0500, Garrett Rooney wrote:
> This part in particular seems to break the build of the bindings on OS
> X. -shared is not the correct flag on all platforms. By removing that
> line and by changing the invocation of apr_config to include the
> --link-ld line I can at least get the bindings to compile on OS X.

--link-ld looks to be necessary in some cases because of APR use.
--shared should be unnecessary, perl will give us that automatically or
whatever the proper option is on your platform.

> The use of LD_LIBRARY_PATH here seems kind of bogus as well, as that
> isn't the correct environment variable to set on all platforms.

We can get the proper name of it from apr_config with:
apr--confing --shlib-path-var
 
> Either way, I'm not sure what the correct fix is, but this is the
> current patch I'm using.

What you submitted looks good to me.

-- 
Ben Reser <ben@reser.org>
http://ben.reser.org
"Conscience is the inner voice which warns us somebody may be looking."
- H.L. Mencken
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org