[SOLUTION] Re: svn commit: r358411 - head/contrib/sendmail/src

[Jung-uk Kim]

On 20. 3. 4., Eugene Grosbein wrote:
> 04.03.2020 0:10, John Baldwin wrote:
> 
>> When the handbook text was first written we didn't have an OpenSSL in ports that would conflict
> 
> Thank you very much for this key for solving the problem.
> This occured to be missing bit of information I was looking for.
> 
> So, the fix is simple. We need only two things: first, base sendmail must be linked with base libcrypto
> and second, cyrus-sasl2 must be linked with base libcrypto too,
> despite of presence of openssl from ports, as an option.
> 
> First goal is accomplished by replacing SENDMAIL_LDFLAGS=-L/usr/local/lib
> with SENDMAIL_LDADD=/usr/local/lib/libsasl2.so in /etc/make.conf
> and the Handbook needs this change.
> 
> Second goal is accomplished with r515574 for ports that added new option (disabled by default).
> If enabled, it builds the port using base openssl and libcrypto:
> https://svnweb.freebsd.org/ports?view=revision&revision=515574

The canonical way of overriding ssl version is adding
"DEFAULT_VERSIONS+=ssl=<openssl variant>" to /etc/make.conf.  Please see
ports/Mk/Uses/ssl.mk.  If you want to add a new method to override it
per port, please write a patch for ssl.mk and submit it to portmgr
(maintainer).  There's no reason to hack every single port.

> Unfortunately, it was back out by maintainer (ume) in r527347 as
> "wrong". It was not.
>
> Dear maintainer, please consider applying r515574 back.
> Feel free renaming port option and improving the description, if you
> wish because I cannot think up better version.
>
> New option for cyrus-sasl2 should be mentioned in the Handbook, too.
>
> With these two changes source upgrade path for such stable/11 system
> is unbroken.

I am not the maintainer but why did you do it without maintainer's
approval in the first place? :-(

Jung-uk Kim