Created attachment 192176 [details] svn diff for security/stunnel ``` security/stunnel: Fix build with LibreSSL 2.7 - LibreSSL 2.7 implements most of OpenSSL 1.1 API PR: Approved by: (maintainer) ```
Isn't this more or less a duplicate of #224148?
Created attachment 192256 [details] svn diff for security/stunnel Patch was incomplete I'd say they're not at all the same. There's only one change to the actual code in src/ctx.c for something LibreSSL does not and will not have. The other patch overwrites the OPENSSL_VERSION_NUMBER that is set by LibreSSL to make stunnel happy. All in all a far smaller change to the port with far fewer opportunity to cause conflict with stunnel updates.
(In reply to Bernard Spil from comment #2) Looks like the same sort of patch to me. Will you guys please update this to use autoconf magic so that we can submit this upstream and have them commit it?
From a good while ago. Upstream will not include anything LibreSSL related. > -------- Original Message -------- > Subject: Re: Patch for stunnel to build with LibreSSL > Date: 2015-06-02 10:32 > From: Michal Trojnara <Michal.Trojnara@mirt.net> > To: Bernard Spil <spil.oss@gmail.com> > > Hi Bernard, > > This is a very nice and clean patch indeed. > I would use it if I ever decided to support LibreSSL. > > Best regards, > Mike This is the cleanest patch I can create for stunnel. As and when I update security/libressl to 2.7 branch the patch can be further simplified. I believe this patch is extremely unlikely to cause conflicts on future versions of stunnel and would be happy to fix them if they would. Cheers, Bernard.
(In reply to Bernard Spil from comment #4) Which is why I recommended using autoconf to detect the missing functions and have it not directly be related to libressl. Please see the previous discussion on this in #224148.
*** This bug has been marked as a duplicate of bug 224148 ***