<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Sat, Oct 5, 2013 at 2:14 PM, Steve Borho <span dir="ltr"><<a href="mailto:steve@borho.org" target="_blank">steve@borho.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote"><div class="im">On Sat, Oct 5, 2013 at 1:00 PM, Derek Buitenhuis <span dir="ltr"><<a href="mailto:derek.buitenhuis@gmail.com" target="_blank">derek.buitenhuis@gmail.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div>On 10/4/2013 12:22 AM, Derek Buitenhuis wrote:<br>

> Queued to test tomorrow morn.<br></div></blockquote><div><br></div></div><div>Thanks for testing</div><div class="im"><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">

Findings:<br>
t<br>
Lib links via:<br>
<br>
/usr/bin/c++  -fPIC  -fPIC -O3 -DNDEBUG  -shared -Wl,-soname,libx265.so -o libx265.so CMakeFiles/x265.dir/dllmain.cpp.o common/libcommon.a encoder/libencoder.a common/vec/libPrimitivesVec.a common/x86/libPrimitivesASM.a -lpthread -lm -lrt<br>

</blockquote><div><br></div></div><div>this looks somewhat sane, ignoring the redundant -fPIC.  Though I wonder if the link order is wrong for GCC.  It might be discarding much of libcommon.a because it is listed before libencoder.a</div>
</div></div></div></blockquote><div><br></div><div>I found that the second patch was unnecessary once I changed the order of the libraries, and now the shared library is much larger than the executable so I've pushed the first patch with those changes.</div>
<div><br></div><div>I couldn't figure out why cmake is linking the static libs into the executable, but it didn't seem like a problem that should hold up shared library support.</div><div><br></div><div>Let me know if you run into any further problems with it.</div>
<div><br></div><div>Thanks</div><div><br></div><div>--</div><div>Steve Borho</div></div></div></div>