brew info llvm
fails when using homebrew/brew
#796
Comments
On Linuxbrew/brew, we stub out brew/Library/Homebrew/os/linux.rb Lines 22 to 28 in 27df536
We could try sending that patch upstream to Homebrew/brew. It seems dirty though. Better I think would be to patch the formulae to guard OS::Mac::CLT.installed? with if/unless OS.mac? . There are 29 such formulae that also have Linux bottles.
For example… diff --git a/Formula/llvm.rb b/Formula/llvm.rb
index c7bdbedb2f..7f354713ff 100644
--- a/Formula/llvm.rb
+++ b/Formula/llvm.rb
@@ -256,7 +256,7 @@ class Llvm < Formula
end
def build_libcxx?
- build.with?("libcxx") || !MacOS::CLT.installed?
+ build.with?("libcxx") || (OS.mac? && !MacOS::CLT.installed?)
end
# Clang cannot find system headers if Xcode CLT is not installed |
I have a list of 77 formula that fail |
There's a lot of |
There's a few formulae that have |
I just did a blanket change with |
I think depends on Xcode does not imply Xcode is needed on Linux. It just means that you need Xcode on Mac. |
Good job, everyone! 🎉 |
brew info llvm
fails with:The text was updated successfully, but these errors were encountered: