Skip to content

Commit

Permalink
msmtp: Explicitly request openssl
Browse files Browse the repository at this point in the history
This is a preventative measure.

Msmtp can be built with either gnutls or openssl as its security lib.
By default, it will use gnutls if it exists on the user's system.  It
will use openssl only if either the user requests it or gnutls is
absent.  The current formula is silent about what security lib to use,
which means the brewer will get gnutls if it's present.

This is risky, because brewers have no idea that this is the case.
Suppose a brewer has gnutls brewed, and then brews msmtp with the
current official formula.  Then suppose the brewer removes gnutls later
but keeps msmtp.  Then msmtp will break, the user will have no idea why,
there probably will be a ticket, and no matter what the brewer will have
to rebrew.

It therefore seems better to explicitly ask msmtp to build against
openssl, since that isn't transient.  Msmtp will find the system's ssl.

For users who never brewed msmtp with gnutls already on their machines,
this commit changes nothing at all.

Closes Homebrew#20756.

Signed-off-by: Adam Vandenberg <[email protected]>
  • Loading branch information
chdiza authored and adamv committed Jun 25, 2013
1 parent 348c231 commit da7be61
Showing 1 changed file with 5 additions and 0 deletions.
5 changes: 5 additions & 0 deletions Library/Formula/msmtp.rb
Original file line number Diff line number Diff line change
Expand Up @@ -10,9 +10,14 @@ class Msmtp < Formula
# msmtp enables OS X Keychain support by default, so no need to ask for it.

def install
# Msmtp will build against gnutls by default if it exists on the
# system. This sets up problems if the user later removes gnutls.
# So explicitly ask for openssl, and ye shall receive it whether
# or not gnutls is present.
args = %W[
--disable-dependency-tracking
--prefix=#{prefix}
--with-ssl=openssl
]

system "./configure", *args
Expand Down

0 comments on commit da7be61

Please sign in to comment.