Skip to contentSkip to conversation

msmtp: fix configure with openssl/gnutls

PATCH New Ryan Coe 2 participants 3 messages Dec 29, 2018 – Dec 30, 2018

lore
Share & tools
lore

Ryan Coe

author
#0 lore raw

PATCH 1/1 msmtp: fix configure with openssl/gnutls

Msmtp no longer uses openssl as default [1]. The configure options have changed from --use-ssl to --use-tls. As a note, openssl usage is discouraged and may be removed in the future.

[1] https://marlam.de/msmtp/news/openssl-discouraged/

---
 package/msmtp/msmtp.mk | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)
package/msmtp/msmtp.mk +3 -3
index a495072abf..a26de92ef9 100644
@@ -41,17 +41,17 @@ MSMTP_CONF_OPTS += --without-libsecret
 endif
 
 ifeq ($(BR2_PACKAGE_OPENSSL),y)
-MSMTP_CONF_OPTS += --with-ssl=openssl
+MSMTP_CONF_OPTS += --with-tls=openssl
 MSMTP_DEPENDENCIES += openssl
 ifeq ($(BR2_STATIC_LIBS),y)
 # openssl uses zlib, so we need to explicitly link with it when static
 MSMTP_CONF_ENV += LIBS=-lz
 endif
 else ifeq ($(BR2_PACKAGE_GNUTLS),y)
-MSMTP_CONF_OPTS += --with-ssl=gnutls
+MSMTP_CONF_OPTS += --with-tls=gnutls
 MSMTP_DEPENDENCIES += gnutls
 else
-MSMTP_CONF_OPTS += --with-ssl=no
+MSMTP_CONF_OPTS += --with-tls=no
 endif
 
 $(eval $(autotools-package))
Signature · 1 line
2.20.1
#1 lore raw

PATCH 1/1 msmtp: fix configure with openssl/gnutls

Hello Ryan,

Thanks for this patch.

Quoted 3 lines · Ryan Coe

Msmtp no longer uses openssl as default [1]. The configure options have changed from --use-ssl to --use-tls. As a note, openssl usage is discouraged and may be removed in the future.

Then perhaps, as a separate patch, you could invert the gnutls/openssl preference, so that if both are available, gnutls is used in favor of openssl ?

Best regards,

Thomas

Signature · 3 lines
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
#2 lore raw

PATCH 1/1 msmtp: fix configure with openssl/gnutls

In reply to Ryan Coe

Hello,

Quoted 10 lines · Ryan Coe

Msmtp no longer uses openssl as default [1]. The configure options have changed from --use-ssl to --use-tls. As a note, openssl usage is discouraged and may be removed in the future.

[1] https://marlam.de/msmtp/news/openssl-discouraged/

---
 package/msmtp/msmtp.mk | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

I've reworked the commit log to indicate since when msmtp has changed this configure option: it is quite important to know for our stable/LTS branches. Also, I've dropped the indication that openssl is no longer the default: it is unrelated to this specific patch.

Applied with those changes. Thanks!

Thomas

Signature · 3 lines
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com