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
rubocops: Permit uses_from_macos "cyrus-sasl" #10808
rubocops: Permit uses_from_macos "cyrus-sasl" #10808
Conversation
Review period will end on 2021-03-09 at 23:15:46 UTC. |
https://github.com/Homebrew/brew/pull/10808/checks?check_run_id=2061171033#step:14:46 |
6dcf4e7
to
8f13fd2
Compare
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.
Debian calls this libsasl2, so let's just discuss the name first.
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.
I'm OK with the name.
|
Review period ended. |
8f13fd2
to
015e27c
Compare
This PR is gated on |
macOS provides /usr/lib/libsasl2.dylib
015e27c
to
0bba3bc
Compare
macOS provides
/usr/lib/libsasl2.dylib
brew style
with your changes locally?brew typecheck
with your changes locally?brew tests
with your changes locally?macOS Catalina 10.15 provides
/usr/lib/libsasl2.dylib
. I have not checked other versions of macOS.See the companion PR Homebrew/homebrew-core#72759
cyrus-sasl 2.1.27 (new formula)