Brendonovich / swift-rs

Call Swift functions from Rust with ease!
Apache License 2.0
246 stars 28 forks source link

Force using system clang #50

Closed lucasfernog closed 11 months ago

lucasfernog commented 1 year ago

We're currently spawning the clang command here. If the PATH is modified (mine was export PATH="/opt/homebrew/opt/llvm/bin:$PATH", clang will actually point to a different version of clang which does not include libclang_rt.ios.a or libclang_rt.iossim.a.

What do you think about changing Command::new("clang") to Command::new("/usr/bin/clang") so it always use the clang we're expecting?

Brendonovich commented 1 year ago

Idk anything about this but if we can guarantee that /usr/bin/clang always exists then this sounds fine