Re: MSVC Build support with visual studio 2019

Поиск
Список
Период
Сортировка
От Juanjo Santamaria Flecha
Тема Re: MSVC Build support with visual studio 2019
Дата
Msg-id 155847452482.1600.2512032648407115756.pgcf@coridan.postgresql.org
обсуждение исходный текст
Ответ на Re: MSVC Build support with visual studio 2019  (Haribabu Kommi <kommi.haribabu@gmail.com>)
Ответы Re: MSVC Build support with visual studio 2019  (Haribabu Kommi <kommi.haribabu@gmail.com>)
Список 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 по дате отправления:

Предыдущее
От: Peter Geoghegan
Дата:
Сообщение: Re: PG 12 draft release notes
Следующее
От: Tom Lane
Дата:
Сообщение: Re: pgindent run next week?