Re: MSVC Build support with visual studio 2019 - Mailing list pgsql-hackers

From Juanjo Santamaria Flecha
Subject Re: MSVC Build support with visual studio 2019
Date
Msg-id 155847452482.1600.2512032648407115756.pgcf@coridan.postgresql.org
Whole thread Raw
In response to Re: MSVC Build support with visual studio 2019  (Haribabu Kommi <kommi.haribabu@gmail.com>)
Responses Re: MSVC Build support with visual studio 2019  (Haribabu Kommi <kommi.haribabu@gmail.com>)
List pgsql-hackers
I have gone through path '0001-Support-building-with-visual-studio-2019.patch' only, but I am sure some comments will
alsoapply to back branches.
 

1. The VisualStudioVersion value looks odd:

+    $self->{VisualStudioVersion}        = '16.0.32.32432';

Are you using a pre-release version [1]?

2. There is a typo: s/stuido/studio/:

+    # The major visual stuido that is suppored has nmake version >= 14.20 and < 15.

There is something in the current code that I think should be also updated. The code for _GetVisualStudioVersion
contains:

  if ($major > 14)
     {
     carp
      "The determined version of Visual Studio is newer than the latest supported version. Returning the latest
supportedversion instead.";
 
     return '14.00';
     }

Shouldn't the returned value be '14.20' for Visual Studio 2019?

Regards,

Juan José Santamaría Flecha

[1] https://docs.microsoft.com/en-us/visualstudio/releases/2019/history#release-dates-and-build-numbers

pgsql-hackers by date:

Previous
From: Peter Geoghegan
Date:
Subject: Re: PG 12 draft release notes
Next
From: Tom Lane
Date:
Subject: Re: pgindent run next week?