Use an older version of binutils on Mac, required to produce valid libraries.

Author: jonsimantov

.github/workflows/cpp-packaging.yml

@@ -22,7 +22,9 @@ on:
 env:
   # Packaging prerequisites
   # Binutils 2.35.1 released Sep 19, 2020
-  binutilsVer: "2.35.1"
+  binutilsVerLinux: "2.35.1"
+  # On Mac, use an older Binutils to avoid corrupted libraries.
+  binutilsVerDarwin: "2.28.1"
   # Demumble 1.1.0 released Nov 13, 2018
   demumbleVer: "1.1.0"
   # Use SHA256 for hashing files.
@@ -67,8 +69,10 @@ jobs:
         include:
         - os: ubuntu-latest
           tools_platform: linux
+          binutils_version: ${{ env.binutilsVerLinux }}
         - os: macos-latest
           tools_platform: darwin
+          binutils_version: ${{ env.binutilsVerDarwin }}
     steps:
       - name: setup Xcode version (macos)
         if: runner.os == 'macOS'
@@ -81,13 +85,13 @@ jobs:
           # Github runners.
           for retry in {1..10} error; do
           if [[ $retry == "error" ]]; then exit 5; fi
-            curl -L https://ftpmirror.gnu.org/binutils/binutils-${{ env.binutilsVer }}.tar.xz --output binutils.tar.xz && break
+            curl -L https://ftpmirror.gnu.org/binutils/binutils-${{ matrix.binutils_version }}.tar.xz --output binutils.tar.xz && break
             sleep 300
           done
           set -e
 
           tar -xf binutils.tar.xz
-          mv ./binutils-${{ env.binutilsVer }} ./binutils-src
+          mv ./binutils-${{ matrix.binutils_version }} ./binutils-src
           cd binutils-src
           ./configure --enable-targets=all --prefix=/tmp/binutils
           make

CMakeLists.txt

@@ -250,7 +250,11 @@ if(DESKTOP)
 
   add_external_library(uWebSockets)
 
-  set(websockets_additional_defines "")
+  # Binutils on Mac doesn't support thread-local storage (required by
+  # websockets), but because we only use websockets via the scheduler,
+  # we don't need it.
+  
+  set(websockets_additional_defines "-D__thread=")
 
   # uWebSockets does not come with a CMakeLists file, so define the target.
   # Note that since it depends on OpenSSL, only do so if that was found.