Skip to content

Commit d654ce2

Browse files
fhgwrightherbygillot
authored andcommitted
alpine: Require curl-ca-bundle with +ssl.
Some server certificates aren't based on root CAs included with the OS (at least for some OS versions). Installing curl-ca-bundle typically provides the needed root CAs, and isn't very expensive. Since other common installs already depend on curl-ca-bundle, it's likely that Alpine has been opportunistically getting it in most cases. Theoretically, this change should get a revbump to ensure that the dependency is included. But since most installations probably already have curl-ca-bundle installed, it's not really worth a rebuild to fix the rare cases. Also removes redundant "platforms darwin" (to get rid of a lint warning). TESTED: Correctly shows dependency iff +ssl. Active curl-ca-bundle fixes access to at least one server.
1 parent 735a9b9 commit d654ce2

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

mail/alpine/Portfile

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,6 @@ revision 0
88
categories mail
99
license Apache-2
1010
maintainers netpurgatory.com:john gmail.com:jerryyhom openmaintainer
11-
platforms darwin
1211

1312
homepage https://alpineapp.email/
1413
description alpine - a Program for Internet News and Email
@@ -80,6 +79,8 @@ variant ldap description {LDAP support} {
8079

8180
variant ssl description {OpenSSL support} {
8281
depends_lib-append path:lib/libssl.dylib:openssl
82+
# Some server certificates may require root CAs not provided by the OS
83+
depends_run-append port:curl-ca-bundle
8384
configure.args-delete --without-ssl
8485
build.env-delete SSLTYPE=none
8586
}

0 commit comments

Comments
 (0)