Only enable Metal Pipeline if the Metal framework is found

H-uru · Jun 2, 2024 · 1826acf · 1826acf
1 parent 2627724
commit 1826acf
Show file tree

Hide file tree

Showing 3 changed files with 20 additions and 2 deletions.
diff --git a/CMakeLists.txt b/CMakeLists.txt
@@ -124,6 +124,7 @@ find_package(ZLIB REQUIRED)
 
 if(APPLE)
  find_package(Security)
+ find_package(Metal)
 elseif(UNIX)
  find_package(LIBSECRET)
  find_package(Uuid REQUIRED)
@@ -147,7 +148,7 @@ endif(PLASMA_EXTERNAL_RELEASE)
 # Pipeline Renderers
 cmake_dependent_option(PLASMA_PIPELINE_DX "Enable DirectX rendering pipeline?" ON "DirectX_FOUND" OFF)
 cmake_dependent_option(PLASMA_PIPELINE_GL "Enable OpenGL rendering pipeline?" ON "TARGET epoxy::epoxy" OFF)
-cmake_dependent_option(PLASMA_PIPELINE_METAL "Enable Metal rendering pipeline?" ON "APPLE" OFF)
+cmake_dependent_option(PLASMA_PIPELINE_METAL "Enable Metal rendering pipeline?" ON "TARGET Metal::Metal" OFF)
 
 if(PLASMA_PIPELINE_DX)
  add_definitions(-DPLASMA_PIPELINE_DX)

diff --git a/Sources/Plasma/FeatureLib/pfMetalPipeline/CMakeLists.txt b/Sources/Plasma/FeatureLib/pfMetalPipeline/CMakeLists.txt
@@ -49,7 +49,7 @@ target_link_libraries(pfMetalPipeline
  CoreLib
  pnNucleusInc
  plPipeline
- "-framework Metal"
+ Metal::Metal
  "-framework MetalPerformanceShaders"
  PRIVATE
  plStatusLog

diff --git a/cmake/FindMetal.cmake b/cmake/FindMetal.cmake
@@ -0,0 +1,17 @@
+include(FindPackageHandleStandardArgs)
+
+find_library(
+ Metal_LIBRARY
+ NAMES Metal
+)
+
+find_package_handle_standard_args(Metal REQUIRED_VARS Metal_LIBRARY)
+
+if(Metal_FOUND AND NOT TARGET Metal::Metal)
+ add_library(Metal::Metal INTERFACE IMPORTED)
+ set_target_properties(
+ Metal::Metal PROPERTIES
+ INTERFACE_LINK_LIBRARIES ${Metal_LIBRARY}
+ )
+endif()
+