<div dir="ltr">Hi, I have been gone for the past 5 weeks, but I have an unfinished patch which will make Interpreter independent from Python so you can build the interpreter unittests regardless of your setting of LLDB_DISABLE_PYTHON.  This actually makes more sense, because even without my patch Python is only a small subset of the functionality contained in Interpreter, and the other stuff should still be able to be unit tested.</div><br><div class="gmail_quote"><div dir="ltr">On Tue, Jul 14, 2015 at 1:52 AM Pavel Labath <<a href="mailto:labath@google.com">labath@google.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">labath added a comment.<br>
<br>
In <a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__reviews.llvm.org_D11127-23203885&d=AwMFaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=MEqT8U_n7oNfuDW5NRbY3ZV384ZquXIYFPWmprwUdKM&m=6Tt5NhtYbd3GeioaoSxSGhCRnxsqNi9sWb3I8Zn6ebA&s=BIEnnQr1XujNXsW1rMGR_ON6Ci1dJIEt7pNVu8_qeYA&e=" rel="noreferrer" target="_blank">http://reviews.llvm.org/D11127#203885</a>, @loladiro wrote:<br>
<br>
> I was getting a link error with python. I just assumed it was required since it was explicitly linked. Do you mean remove it entirely, independent of whether LLDB_DISABLE_PYTHON is set, or just in that case?<br>
<br>
<br>
I am pretty sure that the dependency is required if the whole lldb is built with python. I am proposing we make the just the dependency on python conditional on the LLDB_DISABLE_PYTHON setting. Could you check if that works for you?<br>
<br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__reviews.llvm.org_D11127&d=AwMFaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=MEqT8U_n7oNfuDW5NRbY3ZV384ZquXIYFPWmprwUdKM&m=6Tt5NhtYbd3GeioaoSxSGhCRnxsqNi9sWb3I8Zn6ebA&s=3hUys2Eq6hsJv05EfBl1W4qPNAWQlf14c1_w0M61F8M&e=" rel="noreferrer" target="_blank">http://reviews.llvm.org/D11127</a><br>
<br>
<br>
<br>
</blockquote></div>