Re: Fixing build of MSVC with OpenSSL 3.0.0
Michael Paquier <michael@paquier.xyz>
From: Michael Paquier <michael@paquier.xyz>
To: Daniel Gustafsson <daniel@yesql.se>
Cc: Postgres hackers <pgsql-hackers@lists.postgresql.org>
Date: 2021-10-19T10:52:08Z
Lists: pgsql-hackers
On Tue, Oct 19, 2021 at 10:34:10AM +0200, Daniel Gustafsson wrote: > I think we can tighten the check for GetOpenSSLVersion() a bit since we now now > the range of version in the 1.x.x series. For these checks we know we want > 1.1.x or 3.x.x, but never 2.x.x etc. > > How about the (untested) attached which encodes that knowledge, as well as dies > on too old OpenSSL versions? One assumption hidden behind the scripts of src/tools/msvc/ is that we have never needed to support OpenSSL <= 1.0.1 these days (see for example HAVE_X509_GET_SIGNATURE_NID always set to 1, introduced in 1.0.2) because the buildfarm has no need for it and there is no MSI for this version for years (except if compiling from source, but nobody would do that for an older version anyway with their right mind). If you try, you would already get a compilation failure pretty quickly. So I'd rather keep the code as-is and not add the extra sudden-death check. Now that's only three extra lines, so.. -- Michael
Commits
-
Fix build of MSVC with OpenSSL 3.0.0
- d581960dfbea 9.6.24 landed
- 922e3c3b72d8 10.19 landed
- e00d45fea15f 11.14 landed
- 1539e0ecd6a4 12.9 landed
- abb9ee92c507 13.5 landed
- 81aefaea8293 14.1 landed
- 41f30ecc29c8 15.0 landed