Bump MIN_WINNT for MINGW to clear a build error

Because we have been setting this too low, there has been a
long-standing warning about a missing declaration for inet_pton().
Modern gcc now considers this an error, so we have been getting failures
on the buildfarm animal fairywren.

Fix suggested by Thomas Munro.

This isn't needed in later branches, as they already set MIN_WINNT
higher, nor on earlier branches because they don't use inet_pton().

Discussion: https://postgr.es/m/574fae43-c993-4a25-b0e5-04c3e9c36d6d@dunslane.net

diff --git a/src/include/port/win32.h b/src/include/port/win32.h
index c6213c77c3aee8e239de6a7c2ec0fed5140ef9ae..4b180bf30d247d7b0b3a749185049ff2d9becf54 100644 (file)
--- a/src/include/port/win32.h
+++ b/src/include/port/win32.h
@@ -13,10 +13,12 @@
  * Make sure _WIN32_WINNT has the minimum required value.
  * Leave a higher value in place. When building with at least Visual
  * Studio 2015 the minimum requirement is Windows Vista (0x0600) to
- * get support for GetLocaleInfoEx() with locales. For everything else
+ * get support for GetLocaleInfoEx() with locales. For other MSC versions
  * the minimum version is Windows XP (0x0501).
+ * For non-MSC compilers use 0x0600, which is required to get the inet_pton()
+ * declaration.
  */
-#if defined(_MSC_VER) && _MSC_VER >= 1900
+#if !defined(_MSC_VER) || _MSC_VER >= 1900
 #define MIN_WINNT 0x0600
 #else
 #define MIN_WINNT 0x0501