-
-
Notifications
You must be signed in to change notification settings - Fork 12.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
folly, fizz, wangle: build on Mojave #77819
Conversation
This was previously blocked by `folly` not building on Mojave, but no longer.
depends_on "openssl@1.1" | ||
depends_on "snappy" | ||
depends_on "xz" | ||
depends_on "zstd" | ||
|
||
on_macos do | ||
depends_on "llvm" if DevelopmentTools.clang_build_version <= 1100 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tried to make this build-only, but that doesn't work.
Still want to check |
This was previously blocked by `folly` not building on Mojave, but no longer.
def install | ||
on_macos do | ||
ENV.llvm_clang if DevelopmentTools.clang_build_version <= 1100 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also tried building with gcc
, since that's the fallback for fails_with :clang
, but that doesn't work since boost
is built using clang
.
🤖 A scheduled task has triggered a merge. |
brew install --build-from-source <formula>
, where<formula>
is the name of the formula you're submitting?brew test <formula>
, where<formula>
is the name of the formula you're submitting?brew audit --strict <formula>
(after doingbrew install <formula>
)?I haven't been able to test this since I don't have anything running Mojave, but this seems to have worked elsewhere.