From: | Andrew Dunstan <andrew(dot)dunstan(at)2ndquadrant(dot)com> |
---|---|
To: | Michael Paquier <michael(at)paquier(dot)xyz>, Peifeng Qiu <pqiu(at)pivotal(dot)io> |
Cc: | pgsql-hackers(at)postgresql(dot)org, Haribabu Kommi <kommi(dot)haribabu(at)gmail(dot)com> |
Subject: | Re: Compile from source using latest Microsoft Windows SDK |
Date: | 2019-07-24 14:38:47 |
Message-ID: | 22f158ce-3bdd-c7a6-00d6-e870c332e0bb@2ndQuadrant.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On 7/22/19 4:23 AM, Michael Paquier wrote:
> On Mon, Jul 22, 2019 at 04:01:46PM +0800, Peifeng Qiu wrote:
>>> but it's really only a major issue for VS2019
>> VS2019 will use the latest v10 SDK by default. So no need to install 8.1
>> for VS2019.
> Yes, FWIW, I have tested with VS2019 when committing 2b1394f, and in
> this case only the v10 SDK got installed, with no actual issues
> related to the dependency of the SDK reported. In this case I have
> installed VS using the community installer provided by Microsoft.
>
>>> I guess we might need a test for what SDK is available?
>> We can just use the WindowsSDKVersion environment variable to
>> determine the SDK for current cmd session. It's set when you start
>> the Visual Studio Prompt or call one bat script. Developers can
>> choose the right version best suit their need. Detecting all
>> installed SDK version can be done with some registry magic but I
>> think that's not necessary in this case.
> This looks more sensible to do if the environment variable is
> available. Looking around this variable is available when using the
> command prompt for native tools. So using it sounds like a good idea
> to me if it exists.
Yeah, on consideration I think Peifeng's patch upthread looks OK.
(Incidentally, this variable is not set in the very old version of VC
running on currawong).
cheers
andrew
--
Andrew Dunstan https://www.2ndQuadrant.com
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
From | Date | Subject | |
---|---|---|---|
Next Message | Merlin Moncure | 2019-07-24 14:40:06 | double free in ExecHashJoin, 9.6.12 |
Previous Message | Andrew Dunstan | 2019-07-24 14:00:34 | Re: initdb recommendations |