On Fri, Aug 25, 2017 at 11:27 PM, Christian Ullrich <ch...@chrullrich.net> wrote:
> * On 2017-06-21 02:06, Haribabu Kommi wrote: > > Thanks for the review. Here I attached an updated patch with README update. >> > > Hello, > Thanks for the review. > the most recent update to VS 2017, version 15.3, now identifies as "14.11" > rather than "14.10" in the output of nmake /?. Simply adding this value to > the two places that check for 14.10 in your patch appears to work for me. > VS 2017 doesn't change the nmake version to 15, and it is updating with every minor version, so I changed the check to accept everything that is greater than 14.10 and eq 15, in case in future if VS 2017 changes the version number. > In a newly created project, PlatformToolset is still "v141". ToolsVersion > is "15.0" whereas your patch uses "14.1". > > ISTM that the ToolsVersion has been like this in all versions of VS 2017; > in my collection of .vcxproj files the auto-generated PostgreSQL projects > are the only ones using "14.1". > Updated the Tools version to 15.0 and kept the platform toolset as V141, this because the toolset is version is still points to V141, when I create a sample project with VS 2017 and the version number is inline with nmake version also. Regards, Hari Babu Fujitsu Australia
0001-vs-2017-support_v2.patch
Description: Binary data
-- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers