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

link: don't link openssl. #597

Merged
merged 1 commit into from Jul 29, 2016
Jump to file or symbol
Failed to load files and symbols.
+10 −1
Diff settings

Always

Just for now

link: don't link /usr/local openssl.

  • Loading branch information...
MikeMcQuaid committed Jul 28, 2016
commit 01152aec00826dbf84b3c64df77d9a20e79187a7
@@ -24,7 +24,16 @@ def link
mode.dry_run = true if ARGV.dry_run?
ARGV.kegs.each do |keg|
if keg.linked?
if HOMEBREW_PREFIX.to_s == "/usr/local" && keg.name == "openssl"
opoo <<-EOS.undent
Refusing to link: openssl
Linking keg-only OpenSSL means you may end up linking against the insecure,
deprecated system version while using the headers from the Homebrew version.
Instead, pass the full include/library paths to your compiler e.g.:
-I#{HOMEBREW_PREFIX}/opt/openssl/include -L#{HOMEBREW_PREFIX}/opt/openssl/lib
EOS
next
elsif keg.linked?
opoo "Already linked: #{keg}"
puts "To relink: brew unlink #{keg.name} && brew link #{keg.name}"
next
ProTip! Use n and p to navigate between commits in a pull request.