From: | Juanjo Santamaria Flecha <juanjo(dot)santamaria(at)gmail(dot)com> |
---|---|
To: | pgsql-hackers(at)lists(dot)postgresql(dot)org |
Cc: | Haribabu Kommi <kommi(dot)haribabu(at)gmail(dot)com> |
Subject: | Re: MSVC Build support with visual studio 2019 |
Date: | 2019-05-21 21:35:24 |
Message-ID: | 155847452482.1600.2512032648407115756.pgcf@coridan.postgresql.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
I have gone through path '0001-Support-building-with-visual-studio-2019.patch' only, but I am sure some comments will also apply 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 supported version 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
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2019-05-21 21:46:54 | Re: pgindent run next week? |
Previous Message | Peter Geoghegan | 2019-05-21 21:22:53 | Re: PG 12 draft release notes |