fix(macos): fix boost on macos

LizardByte · Jun 20, 2024 · 0c67e74 · 0c67e74
1 parent 429019e
commit 0c67e74
Show file tree

Hide file tree

Showing 4 changed files with 33 additions and 3 deletions.
diff --git a/cmake/compile_definitions/common.cmake b/cmake/compile_definitions/common.cmake
@@ -125,8 +125,8 @@ include_directories(
         "${CMAKE_SOURCE_DIR}/third-party/moonlight-common-c/enet/include"
         "${CMAKE_SOURCE_DIR}/third-party/nanors"
         "${CMAKE_SOURCE_DIR}/third-party/nanors/deps/obl"
-        ${Boost_INCLUDE_DIRS}
         ${FFMPEG_INCLUDE_DIRS}
+        ${Boost_INCLUDE_DIRS}  # has to be the last one or we runtime error on macOS ffmpeg encoder
 )
 
 list(APPEND SUNSHINE_EXTERNAL_LIBRARIES

diff --git a/cmake/prep/options.cmake b/cmake/prep/options.cmake
@@ -15,7 +15,11 @@ option(SUNSHINE_REQUIRE_TRAY "Require system tray icon. Fail the build if tray r
 
 option(SUNSHINE_SYSTEM_WAYLAND_PROTOCOLS "Use system installation of wayland-protocols rather than the submodule." OFF)
 
-option(BOOST_USE_STATIC "Use static boost libraries." ON)
+if(APPLE)
+    option(BOOST_USE_STATIC "Use static boost libraries." OFF)
+else()
+    option(BOOST_USE_STATIC "Use static boost libraries." ON)
+endif()
 
 option(CUDA_INHERIT_COMPILE_OPTIONS
         "When building CUDA code, inherit compile options from the the main project. You may want to disable this if

diff --git a/packaging/macos/Portfile b/packaging/macos/Portfile
@@ -46,7 +46,8 @@ depends_test             port:doxygen \
 
 configure.args           -DBUILD_WERROR=ON \
                          -DCMAKE_INSTALL_PREFIX=${prefix} \
-                         -DSUNSHINE_ASSETS_DIR=etc/sunshine/assets
+                         -DSUNSHINE_ASSETS_DIR=etc/sunshine/assets \
+                         -DBOOST_USE_STATIC=ON
 
 configure.env-append     BRANCH=@GITHUB_BRANCH@
 configure.env-append     BUILD_VERSION=@BUILD_VERSION@

diff --git a/packaging/sunshine.rb b/packaging/sunshine.rb
@@ -21,9 +21,13 @@ class @PROJECT_NAME@ < Formula
     end
   end
 
+  option "without-dynamic-boost", "Statically link Boost libraries"  # default option
+  option "with-dynamic-boost", "Dynamically link Boost libraries"
+
   depends_on "cmake" => :build
   depends_on "node" => :build
   depends_on "pkg-config" => :build
+  depends_on "icu4c" => :recommended
   depends_on "curl"
   depends_on "miniupnpc"
   depends_on "openssl"
@@ -64,6 +68,27 @@ def install
       -DSUNSHINE_ENABLE_TRAY=OFF
       -DTESTS_ENABLE_PYTHON_TESTS=OFF
     ]
+
+    if build.without? "dynamic-boost"
+      args << "-DBOOST_USE_STATIC=ON"
+      ohai "Statically linking Boost libraries"
+
+      unless Formula["icu4c"].any_version_installed?
+        odie <<~EOS
+          icu4c must be installed to link against static Boost libraries,
+          either install icu4c or use brew install sunshine --with-dynamic-boost instead
+        EOS
+      end
+      ENV.append "CXXFLAGS", "-I#{Formula["icu4c"].opt_include}"
+      icu4c_lib_path = "#{Formula["icu4c"].opt_lib}"
+      ENV.append "LDFLAGS", "-L#{icu4c_lib_path}"
+      ENV["LIBRARY_PATH"] = "#{icu4c_lib_path}"
+      ohai "Linking against ICU libraries at: #{icu4c_lib_path}"
+    else
+      args << "-DBOOST_USE_STATIC=OFF"
+      ohai "Dynamically linking Boost libraries"
+    end
+
     system "cmake", "-S", ".", "-B", "build", *std_cmake_args, *args
 
     cd "build" do