TomBebbington / llvm-rs

LLVM wrappers for Rust
BSD 3-Clause "New" or "Revised" License
68 stars 27 forks source link

Build script behaves incorrectly on Windows #19

Open RReverser opened 8 years ago

RReverser commented 8 years ago

On Windows, LLVM is installed to C:\Program Files (x86)\LLVM by default.

So when build script calls, for example, llvm-config --ldflags, it returns -LIBPATH:C:\Program Files (x86)\LLVM\lib, and then build script performs .split_whitespace() which breaks those paths and compilation fails (UPD: apparently those are just paths for debug output, but gcc fails as well for the same reasons).

Of course, I can workaround this by temporarily pointing PATH to a separate folder with those binaries, but thought it's worth reporting so that this could be fixed for other users as well.

RReverser commented 8 years ago

Of course, I can workaround this by temporarily pointing PATH to a separate folder with those binaries, but thought it's worth reporting so that this could be fixed for other users as well.

In fact, not just pointing, but rebuilding LLVM with different CMAKE_INSTALL_PREFIX, otherwise all llvm-config results won't be updated 😞