Re: MSVC Build support with visual studio 2019
Haribabu Kommi <kommi.haribabu@gmail.com>
From: Haribabu Kommi <kommi.haribabu@gmail.com>
To: Andrew Dunstan <andrew.dunstan@2ndquadrant.com>
Cc: PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>
Date: 2019-03-27T00:42:02Z
Lists: pgsql-hackers
Attachments
- 0001-Support-building-with-visual-studio-2019_v2.patch (application/octet-stream) patch v2-0001
On Wed, Mar 27, 2019 at 3:03 AM Andrew Dunstan < andrew.dunstan@2ndquadrant.com> wrote: > > On 3/20/19 8:36 PM, Haribabu Kommi wrote: > > Hi Hackers, > > > > Here I attached a patch that supports building of PostgreSQL with VS > 2019. > > VS 2019 is going to release on Apr 2nd 2019, it will be good if version > 12 > > supports compiling. The attached for is for review, it may needs some > > updates > > once the final version is released. > > > > Commit d9dd406fe281d22d5238d3c26a7182543c711e74 has reduced the > > minimum visual studio support to 2013 to support C99 standards, > > because of this > > reason, the current attached patch cannot be backpatched as it is. > > > > I can provide a separate back branches patch later once this patch > > comes to a stage of commit. Currently all the supported branches are > > possible to compile with VS 2017. > > > > comments? > > > > > > > I have verified that this works with VS2019. > > > There are a few typos in the comments that need cleaning up. > Thanks for the review. I corrected the typos in the comments, hopefully I covered everything. Attached is the updated patch. Once the final VS 2019, I will check the patch if it needs any more updates. Regards, Haribabu Kommi Fujitsu Australia
Commits
-
Add support for Visual Studio 2019 in build scripts
- d49c127d90f9 9.4.24 landed
- 3337fd192596 9.5.19 landed
- 78aaffd285c4 9.6.15 landed
- 0ce8e49b2243 10.10 landed
- 721963f24ae8 11.5 landed
- f2d069ce95b1 12.0 landed
- 2b1394fc2b52 13.0 landed
-
Support building with Visual Studio 2017
- 19acfd6528bc 9.4.20 cited
- f2ab3898f3a2 11.0 cited
-
Require C99 (and thus MSCV 2013 upwards).
- d9dd406fe281 12.0 cited