|From:||Michael Paquier <michael(at)paquier(dot)xyz>|
|To:||Juan José Santamaría Flecha <juanjo(dot)santamaria(at)gmail(dot)com>|
|Cc:||Sandeep Thakkar <sandeep(dot)thakkar(at)enterprisedb(dot)com>, serpashk(at)gmail(dot)com, pgsql-bugs(at)lists(dot)postgresql(dot)org|
|Subject:||Re: BUG #15789: libpq compilation with OpenSSL 1.1.1b fails on Windows with Visual Studio 2017|
|Views:||Raw Message | Whole Thread | Download mbox | Resend email|
On Mon, Jun 24, 2019 at 01:44:18PM +0200, Juan José Santamaría Flecha wrote:
> The updated patches include:
> 1. Comments for AddLibrary.
> 2. The function GetOpenSSLVersion returns: major, minor and fix numbers.
> 3. The version check will default to 1.1.X behaviour for 1.Y.X
> versions if Y > 0.
Thanks for the new patch set! I have been looking at that in depths
and I have adjusted the whole logic a bit here and there. At the end
I found the logic changed in AddLibrary more confusing because the
debugging suffix may change depending on if we use a release-quality
build or not, so I have kept the original interface, and completed it
with a logic allowing the scripts to grab all the libraries it
expects. This has a small gain when using a non-debug installation as
the library names are the same for Win32 and Win64 for >= 1.1.0.
Also, your patch was not working with other versions of MSVC as the
new routine got stuck into the 2017 class, so I had to move it, and I
found that it was cleaner to just make it return a string made of the
3 first digits and to do direct string comparisons.
Please note that it is not necessary to create versions for the
back-branches yet. If necessary, I'll do that myself, but first let's
make sure that we agree about the shape of the patch for HEAD.
Attached is an updated version which I would be fine to commit. I
have tested it with compilation linking to OpenSSL 1.0.2 and 1.1.0 on
Win32 and the build is able to complete. This applies on HEAD only,
where I have run all my tests. The patch is properly indented.
What do you think?
|Next Message||Amit Langote||2019-06-25 08:56:00||Re: BUG #15724: Can't create foreign table as partition|
|Previous Message||Tom Dunstan||2019-06-25 05:25:03||Re: BUG #15871: Regression in 11.4 altering type on column with an index|