Re: Remove support for Visual Studio 2013

Поиск
Список
Период
Сортировка
От Juan José Santamaría Flecha
Тема Re: Remove support for Visual Studio 2013
Дата
Msg-id CAC+AXB3himFH+-pGRO1cYju6zF2hLH6VmwPbf5RAytF1UBm_nw@mail.gmail.com
обсуждение исходный текст
Ответ на Re: Remove support for Visual Studio 2013  (Michael Paquier <michael@paquier.xyz>)
Ответы Re: Remove support for Visual Studio 2013  (Michael Paquier <michael@paquier.xyz>)
Re: Remove support for Visual Studio 2013  (Michael Paquier <michael@paquier.xyz>)
Список pgsql-hackers

On Wed, May 18, 2022 at 2:27 AM Michael Paquier <michael@paquier.xyz> wrote:

@@ -1757,7 +1757,7 @@ get_collation_actual_version(char collprovider, const char *collcollate)
                             collcollate,
                             GetLastError())));
         }
-        collversion = psprintf("%d.%d,%d.%d",
+        collversion = psprintf("%ld.%ld,%ld.%ld",
                                (version.dwNLSVersion >> 8) & 0xFFFF,
                                version.dwNLSVersion & 0xFF,
                                (version.dwDefinedVersion >> 8) & 0xFFFF,

Is this change still required even if we bump MIN_WINNT to 0x0600 for
all the environments that include win32.h?

Right now we are ifdefing that code out for MinGW, so it's not a visible issue, but it'll be when we do. 
 
At the end, this would
mean dropping support for Windows XP and Windows Server 2003 as
run-time environments as listed in [1], which are not supported
officially since 2014 (even if there have been some patches for
some critical issues).  So I'd be fine to raise the bar for v16~,
particularly as this would allow us to get rid of this code related to
locales.
 
Even Windows Server 2008 [1] is at its End of Life, so this should surprise no one.


Regards,

Juan José Santamaría Flecha
 

В списке pgsql-hackers по дате отправления:

Предыдущее
От: Noah Misch
Дата:
Сообщение: Re: Rewriting the test of pg_upgrade as a TAP test - take three - remastered set
Следующее
От: Alvaro Herrera
Дата:
Сообщение: Re: [PATCH] New [relation] option engine