[opensource-dev] Review Request: SNOW-240: Fix libjson naming madness, for standalone.

Boroondas Gupte sllists at boroon.dasgupta.ch
Sun Dec 19 12:48:34 PST 2010


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://codereview.secondlife.com/r/47/#review54
-----------------------------------------------------------

Ship it!


Looks good.

Like this, it should continue working even if a system-wide installed libjson uses the compiler-version-specific filename, like the gentoo ebuild from Techwolf's portage overlay currently does. I guess all other distributions use the plain name, so once this ebuild has been updated, we can even drop looking for the compiler version specific filename in the standalone case completely.

- Boroondas


On 2010-12-19 09:19:37, Aleric Inglewood wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://codereview.secondlife.com/r/47/
> -----------------------------------------------------------
> 
> (Updated 2010-12-19 09:19:37)
> 
> 
> Review request for Viewer.
> 
> 
> Summary
> -------
> 
> On linux (and remember this is about standalone)
> the libjson packages of distributions don't have this
> complex compiler version baked into their name.
> 
> This patch fixes this issue by first searching for
> libjson_linux-gcc-${_gcc_COMPILER_VERSION}_libmt.so
> and when that fails search for the system package
> library file libjson.so.
> 
> 
> This addresses bug SNOW-240.
>     http://jira.secondlife.com/browse/SNOW-240
> 
> 
> Diffs
> -----
> 
>   indra/cmake/FindJsonCpp.cmake b0689af42a71 
> 
> Diff: http://codereview.secondlife.com/r/47/diff
> 
> 
> Testing
> -------
> 
> It works :p (I have Michelle's debian package libjsoncpp0 installed, which provides /usr/lib/libjson.so).
> 
> 
> Thanks,
> 
> Aleric
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.secondlife.com/pipermail/opensource-dev/attachments/20101219/f83474d3/attachment.htm 


More information about the opensource-dev mailing list