svn commit: r520074 - head/ftp/curl

[Kubilay Kocak]

On 14/12/2019 11:33 am, Jan Beich wrote:
> Kubilay Kocak <koobs at FreeBSD.org> writes:
> 
>> -OPTIONS_DEFAULT=	ALTSVC CA_BUNDLE COOKIES GSSAPI_${${SSL_DEFAULT} == base :?BASE :NONE} DICT FTP GOPHER HTTP HTTP2 IMAP OPENSSL POP3 PROXY RESOLV RTSP SMTP TELNET TFTP THREADED_RESOLVER TLS_SRP
>> +OPTIONS_DEFAULT=	ALTSVC CA_BUNDLE COOKIES GSSAPI_${${SSL_DEFAULT} == base :?BASE :NONE} DICT FTP GOPHER HTTP HTTP2 IMAP NTLM OPENSSL POP3 PROXY RESOLV RTSP SMTP TELNET TFTP THREADED_RESOLVER TLS_SRP
> 
> Did you forget to bump PORTREVISION when changing default options?
> 

The thinking was that: NTLM is (was) an implicit default because OPENSSL 
is an OPTIONS_DEFAULT, and that this fixed a build failure when OpenSSL 
was built without DES (a default option), that it wasn't necessary.

Said another way, that it wasn't necessary because it was a noop except 
for ports users with the build failure

Happy to bump the portrev if there's a case for it though