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

On Nov 29, 2003, at 1:58 PM, Ben Reser wrote:

> 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.

Cool.

>> 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

That looks promising, I'll give it a shot.

>> 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.

Great, once I've played around with the --shlib-path-var stuff I'll
clean this up and commit it.

-garrett

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Sat Nov 29 20:15:07 2003