-
-
Notifications
You must be signed in to change notification settings - Fork 1k
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
Remove incompatible version of openssl098 #3857
Conversation
Check Homebrew requirements https://github.com/Homebrew/homebrew/wiki/Installation" | ||
} | ||
|
||
} |
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.
double closing of the function?
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.
good catch! :)
@@ -290,6 +311,8 @@ requirements_osx_brew_libs_default() | |||
|
|||
requirements_osx_brew_check_custom homebrew/dupes # for: zlib | |||
|
|||
undesired_check openssl098 |
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.
not sure it should be in custom
or rather in before
or define
? or maybe we should add before_install
? or even a new step uninstall
? - if custom
is called always - just leave it :)
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.
Yeah, the requirements_osx_brew_libs_default
is called in few places, that's why I put it there. Not sure if we should have a dedicated uninstall
function. Possibly. Didn't want to invest too much of the time having RVM2 down the road...
…openssl098 # Conflicts: # CHANGELOG.md
Signed-off-by: Piotr Kuczynski <piotr.kuczynski@gmail.com>
# Conflicts: # CHANGELOG.md
Fixes #3844
Uses #3855