<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Oct 23, 2013 at 5:03 AM, 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 class="im">On 10/23/2013 6:16 AM, Steve Borho wrote:<br>

> # HG changeset patch<br>
> # User Steve Borho <<a href="mailto:steve@borho.org">steve@borho.org</a>><br>
> # Date 1382480398 18000<br>
> #      Tue Oct 22 17:19:58 2013 -0500<br>
> # Node ID a8d4928926e82026ed71ca0af210a5f4f16d405d<br>
> # Parent  6d96d64c4e9a2c526b57274760a7147241328cb3<br>
> cmake: generate and install pkgconfig file<br>
><br>
> This installs the x265.pc into /usr/local/lib/pkgconfig/ with these contents:<br>
<br>
</div>s#/usr/local/lib/#/specificed/prefix/#g in the commit message...<br>
<div class="im"><br>
> Name: x265<br>
> Description: H.265/HEVC video encoder<br>
> Version: 0.4.1<br>
> Libs: -L${libdir} -lx265<br>
> Libs.private:  -lstdc++ -lm -lc -lpthread -lrt<br>
<br>
</div>This is not portable between systems, which may not use GNU's<br>
libsdtc++. OS X, can for example, use libc++. Don't get me started<br>
on Libav's MSVC stuff... ;)<br>
<br></blockquote><div><div><br></div><div>-lstdc++ -lm -lc are coming from CMAKE_CXX_IMPLICIT_LIBRARIES. ie these are libraries detected by CMake that all C++ apps need to link with.  It should do the right thing if your compiler / operating system require different libraries, though testing on those platforms would be quite helpful.</div>
</div><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">
Also, I thought you're not supposed to specify -lpthread explicitly --<br>
that is, use -pthread when compiling/linking instead.<br></blockquote><div><br></div><div><div>I didn't know about -pthread.  If that is going to prevent this from working I can add a special case for it in the loop that builds the PRIVATE_LIBS string<br>
</div></div><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">
-lc should not be added.<br></blockquote><div><br></div><div>Will it break things or is it just unnecessary (is it worth the trouble of filtering it out)? </div><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">

Not all systems use librt, and it is also not portable. I think MinGW might not.<br>
<br>
All of these things should be set properly based on configure checks.</blockquote><div><br></div><div>they are, -lrt is only added on UNIX</div><div> </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">
<div class="im">
> The pkg-config file is only generated and installed if CMake finds pkg-config<br>
><br>
> Q1: it seems unwise to hard-code "/lib" in a few places, what if lib64 is<br>
> used or something else?<br>
<br>
</div>Autotools can set --libdir IIRC, which works around it as such:<br>
<br>
${prefix}/${libdir}<br>
<br>
Or at least, it has *something* like this, which I cannot recall the name<br>
of off the top of my head.</blockquote><div><br></div><div>ok</div><div> </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">
<div class="im">
> Q2: Should x265.pc have a version number like x265-0.4.1.pc?<br>
<br>
</div>No. This demeans the entire point of pkg-config. The version is set<br>
in the .pc file, and it is what should be checked by end users, via<br>
pkg-config.</blockquote><div><br></div><div>good </div></div><div><br></div>-- <br>Steve Borho
</div></div>