Re: MSVC Build support with visual studio 2019

From: Michael Paquier <michael(at)paquier(dot)xyz>
To: Haribabu Kommi <kommi(dot)haribabu(at)gmail(dot)com>
Cc: Juan José Santamaría Flecha <juanjo(dot)santamaria(at)gmail(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: MSVC Build support with visual studio 2019
Date: 2019-07-02 05:10:11
Message-ID: 20190702051011.GF1388@paquier.xyz
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Mon, Jul 01, 2019 at 07:56:29PM +1000, Haribabu Kommi wrote:
> We stopped the support of building with all the visual studio versions less
> than 2013. I updated the SDK versions accordingly.

I have spent some time looking around, and wikipedia-sensei has proved
to be helpful to grasp the release references:
https://en.wikipedia.org/wiki/Microsoft_Windows_SDK

So the suggestions from the patch are fine. This one was actually
forgotten:
src/tools/msvc/README:from www.microsoft.com (v6.0 or greater).

> The similar change is required for the CreateProject also.

I have changed both messages so as the version of VS attempted to be
used is reported in the error message directly.

> + # The major visual studio that is supported has nmake
> + # version >= 14.30, so stick with it as the latest version
>
> The major visual studio version that is supported has nmake version
> <=14.30

Damn. Thanks for pointing out that.

> Except for the above two changes, overall the patch is in good shape.

OK, committed to HEAD for now after perltidy'ing the patch. Let's see
what the buildfarm has to say about it first. Once we are sure that
the thing is stable, I'll try to backpatch it. This works on my own
dev machines with VS 2015 and 2019, but who knows what hides in the
shadows...
--
Michael

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Julien Rouhaud 2019-07-02 05:12:38 Re: cleanup & refactoring on reindexdb.c
Previous Message amul sul 2019-07-02 04:47:17 Re: [HACKERS] advanced partition matching algorithm for partition-wise join