Don't use an environment variable, add a command line flag to the script instead <br><div class="gmail_quote"><div dir="ltr">On Fri, Apr 15, 2016 at 3:27 PM Francis Ricci <<a href="mailto:fjricci@fb.com">fjricci@fb.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">fjricci added a comment.<br>
<br>
So I looked at `finishSwigPythonLLDB.py`, and it does look like that's where the bug is. This script sets the end of the symlink path with:<br>
<br>
  strSrc = os.path.join("lib", "liblldb" + strLibFileExtn)<br>
<br>
Note here that "lib" is hardcoded, and doesn't use the `LLVM_LIBDIR_SUFFIX`, which is why the symlink is broken. The only way I can think of to communicate this suffix from cmake into python would be by having cmake set an environment variable (presumably with the same name). Does this seem reasonable, or do you think there's a better solution?<br>
<br>
The solution from `finish-swig-Python-LLDB.sh` would also probably work (enforcing that liblldb.so and python2.7/site-packages/lldb be in the same directory). That solution actually would be more consistent with the way the unit test suite works anyway. It does seem less flexible though.<br>
<br>
<br>
<a href="http://reviews.llvm.org/D19067" rel="noreferrer" target="_blank">http://reviews.llvm.org/D19067</a><br>
<br>
<br>
<br>
</blockquote></div>