Fix build on Wine 6.8 #103

Wine dropped `WINE_NOWINSOCK`, so now we sadly have to define this
globally instead.

CHANGELOG.md

@@ -25,6 +25,11 @@ Versioning](https://semver.org/spec/v2.0.0.html).
   objects on both sides. This greatly reduces the overhead of our VST3 bridging
   by getting rid of all memory allocations during audio processing.
 
+### Fixed
+
+- Fixed builds on Wine 6.8 because of internal changes to Wine's `windows.h`
+  implementation.
+
 ## [3.2.0] - 2021-05-03
 
 ### Added

meson.build

@@ -212,7 +212,11 @@ if with_vst3
   vst3_wine_compiler_options = [
     # Some stuff from `windows.h` results in conflicting definitions
     '-DNOMINMAX',
-    '-DWINE_NOWINSOCK',
+    # Winsock conflicts with the Posix sockets API. Before Wine 6.8 there was a
+    # `WINE_NOWINSOCK` that would exclude just `winsock.h` from `windows.h`, but
+    # they got rid of that so we now need to explicitly define the ifdef guards
+    '-D__WINE_WINSOCKAPI_STDLIB_H',
+    '-D_WINSOCKAPI_',
   ]
   vst3_base_wine_64bit = static_library(
     'vst3_base_wine_64bit',

src/wine-host/bridges/vst2.h

@@ -20,7 +20,8 @@
 
 #ifndef NOMINMAX
 #define NOMINMAX
-#define WINE_NOWINSOCK
+#define __WINE_WINSOCKAPI_STDLIB_H
+#define _WINSOCKAPI_
 #endif
 #include <vestige/aeffectx.h>
 #include <windows.h>

src/wine-host/editor.h

@@ -22,7 +22,8 @@
 
 #ifndef NOMINMAX
 #define NOMINMAX
-#define WINE_NOWINSOCK
+#define __WINE_WINSOCKAPI_STDLIB_H
+#define _WINSOCKAPI_
 #endif
 #include <windows.h>
 #include <function2/function2.hpp>

src/wine-host/utils.h

@@ -25,7 +25,8 @@
 
 #ifndef NOMINMAX
 #define NOMINMAX
-#define WINE_NOWINSOCK
+#define __WINE_WINSOCKAPI_STDLIB_H
+#define _WINSOCKAPI_
 #endif
 #include <windows.h>