<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Sun, Apr 14, 2013 at 12:56 PM, Brecht Van Lommel <span dir="ltr"><<a href="mailto:brechtvanlommel@pandora.be" target="_blank">brechtvanlommel@pandora.be</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">That's seems reasonable, I've attached a patch which tries to<br>
implement it. Could you test if it works on your system?<br></blockquote><div><br></div>I didn't actually test it but it looks like your patch will fail if the static libs are installed and LLVM_STATIC == OFF since it only looks for the dynamic libs if it can't find the static ones.<br>
<br>Attached is a patch that adds a WITH_LLVM_STATIC option (defaults to ON so as to not break the other systems) and only checks
for one of the dynamic/static libs so it'll fail if the correct one isn't
found. Might also be a good idea to do a mark_as_advanced(WITH_LLVM_STATIC) so the win/apple folks aren't tempted to mess with it, dunno?<br></div><div class="gmail_quote"><br></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
Also, just adding the llvm-static package may not be the solution, OSL<br>
needs to be linked with the same static/dynamic library. I'm not sure<br>
how that would be ensured at the level of linux distribution packages.<br></blockquote><br></div><div class="gmail_quote">That's actually how this all started... Fedora was trying to upgrade llvm but my installed OSL version was built against the old version so I had to manually upgrade llvm on the OpenSuse Build Service which then broke building blender due to the llvm-static thing.<br>
</div><div class="gmail_quote"><br></div><div class="gmail_quote"></div><div class="gmail_quote">Dan<br></div></div></div>