msmtp: fix configure with openssl/gnutls
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
@@ -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
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
PATCH 1/1 msmtp: fix configure with openssl/gnutls
In reply to Ryan CoeHello,
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/
Signed-off-by Ryan Coe <[email protected]>--- 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