Bump MIN_WINNT for MINGW to clear a build error

adunstan · adunstan · commit d700e8d75bc3 · 2024-09-30T11:32:32.000-04:00
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
@@ -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
