On Jan 9, 2007, at 08:08, Michael Dunstan wrote:
> On 1/9/07, Carsten Koch <Carsten.Koch icem.com> wrote:
>> Michael Dunstan wrote:
>> ...
>> > And the introduction of the 'rpath' argument
was needed to fix the
>> > following error:
>>
>> at least on my system, g++ does not have an rpath
parameter.
>> Only ld does.
>> And ld does get one via "-Wl,".
>> So my question is: I wonder if that line
>>
>> +LDLIBS=-L%(svn_lib_dir)s
-rpath=/usr/lib:/usr/local/lib -Wl,--
>> rpath -Wl,%(svn_lib_dir)s
>>
>> in your patch should be something like
>>
>> +LDLIBS=-L%(svn_lib_dir)s -Wl,-rpath
-Wl,/usr/lib:/usr/local/lib:%
>> (svn_lib_dir)s
>>
>> Also note that only ld seems to accept the
(undocumented) syntax
>> "--rpath". Other linkers choke on it. So
I suggest to use the
>> slightly more portable syntax "-rpath".
>
> Thanks for the critical eye. I can't say that I had any
particular
> confidence with the position when coping in the rpath
argument.
>
> I just tested your suggestion and it works for me on
FreeBSD.
I've added FreeBSD specific code to setup.py and added the
-rpath
change as well.
committed as r849.
Barry
------------------------------------------------------------
---------
To unsubscribe, e-mail: dev-unsubscribe pysvn.tigris.org
For additional commands, e-mail: dev-help pysvn.tigris.org
|