Brian Lu wrote:
> Alfred Peng wrote:
>   
>> Hi Ginn,
>>
>> Thanks for the comments.
>>
>> The Songbird libraries link against libsqlite indirectly through
>> XULRunner's libxul.so/libxpcom.so (etc.). I think the "$ORIGIN" in
>> RUNPATH should make sure that they won't link to
>> /usr/lib/mps/libsqite3.so in this case. Please correct me if I'm wrong.
>>   
>>     
> '$ORIGIN' only used by in runtime linker and -L/usr/lib/mps will force 
> ld to use /usr/lib/mps/libsqlite3.so
> instead of /usr/lib/libsqlite3.so
>   
Since /usr/lib/songbird/xulrunner isn't available during linking, we
can't put "-L%{_libdir}/songbird/xulrunner" in LDFLAGS. I'll patch the
Songbird's configure to add this.

Thanks,
-Alfred
>> Thanks,
>> -Alfred
>>
>> Ginn Chen wrote:
>>   
>>     
>>> Ah, I see, it's building songbird.mk.
>>>
>>> So I think you should put "-L%{_libdir}/songbird/xulrunner
>>> -R%{_libdir}/songbird/xulrunner" ahead of "-L%{_libdir}/mps
>>> -R%{_libdir}/mps".
>>>
>>> Ginn
>>>
>>> On Jun 11, 2009, at 3:04 PM, Ginn Chen wrote:
>>>
>>>     
>>>       
>>>> Is this necessary? configure should already add nspr/nss CFLAGS/LDFLAGS.
>>>>
>>>> I'm afraid you would link to /usr/lib/mps/libsqiite3.so with this.
>>>>
>>>> Ginn
>>>>
>>>> On Jun 10, 2009, at 11:20 PM, Alfred Peng wrote:
>>>>
>>>>       
>>>>         
>>>>> +%if %option_without_moz_nss_nspr
>>>>> +export CFLAGS="$CFLAGS -I%{_includedir}/mps"
>>>>> +export CXXFLAGS="$CXXFLAGS -I%{_includedir}/mps"
>>>>> +export LDFLAGS="$LDFLAGS -L%{_libdir}/mps -R%{_libdir}/mps
>>>>> -R%{_libdir}/songbird/xulrunner"
>>>>> +%endif
>>>>> +
>>>>>         
>>>>>           
>>   
>>     
>
>   


Reply via email to