slrn: new pull_ssl variant

[niemda (niemda)]

I propose new variant for slrn port which combines existing +ssl and +pull, because existing +pull lacks ssl support. Proposed change to the portfile is:

--- Portfile	2017-01-05 17:04:22.000000000 +0500
+++ /opt/local/var/macports/sources/rsync.macports.org/release/tarballs/ports/news/slrn/Portfile	2017-01-02 05:55:32.000000000 +0500
@@ -34,14 +34,6 @@
 			depends_lib-append	path:lib/libssl.dylib:openssl
 }
 
-# pull_ssl variant
-variant pull_ssl {
-      configure.args-append   --with-slrnpull
-			configure.args-append 	--with-ssl=${prefix} \
-						--with-ssl-includes=${prefix}/include/openssl
-			depends_lib-append	path:lib/libssl.dylib:openssl
-}
-
 variant uudeview description {Use uudeview library to decode uuencoded articles} {
 	# build dependency because uudeview provides only the static library libuu.a
 	depends_build-append	port:uudeview

I am testing slrn built with these options now. My personal preference is to make ssl default option of the port, but this simple change doesn't look very disruptive.

[niemda (niemda)]

Patch for slrn Portfile

[ryandesign (Ryan Carsten Schmidt)]

Your patch is reversed.

Why would we need a new pull_ssl variant? Why can't you just install the port with both of the existing variants together, which should accomplish the same thing?

sudo port install slrn +pull +ssl

[niemda (niemda)]

Yes, did clean and tried installing +pull +ssl again. It worked. I wonder why it doesn't first time. OK, no need to change anything.