<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">
</span>Should be used only for gallium-xlib based glx, since it embeds the<br>
swr/llvmpipe driver.<br></blockquote><div>... <br></div><div>...<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">
</span>There is no LLVM specific code in these - ^^ should not be needed.<br></blockquote><div><br></div><div>Correct.  This was initially to address the problem for OSMesa but I realized it was likely an issue for more than just OSMesa.  After a bit of debugging I see that I was indeed a bit overzealous on that.  Will fix in v2.<br></div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class=""><br>
>> +    if test "x$enable_osmesa$enable_<wbr>gallium_osmesa" != xnono; then<br>
>> +        OSMESA_PC_LIB_PRIV="$OSMESA_<wbr>PC_LIB_PRIV $OSMESA_PC_LIB_PRIV $LLVM_LIBS"<br>
><br>
> I'm assuming the duplicate `$OSMESA_PC_LIB_PRIV` wasn't intended?<br></span></blockquote><div><br></div><div>Will fix in v2.</div><div><br></div><div> <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
These variables have the dependency libs (-lfoo) that the respective<br>
libraries libGL.so/libGLES*so/etc.<br>
Then they are stored in the the .pc Libs.private section - thus anyone<br>
static linking said libraries will reuse it.<br></blockquote><div><br></div><div>This is inded the use case here: building a static libGL or static libOSMesa and having proper dependency info available.</div><div><br></div><div>I'll push a corrected v2 shortly.</div><div><br></div><div>Thanks for the review.<br></div><br></div></div></div>