From dbf47ee55a34b6370f17ff7ed3173fe3e8e5790c Mon Sep 17 00:00:00 2001 From: Jeffrey Morgan Date: Mon, 12 Jan 2026 20:05:11 -0800 Subject: [PATCH] cmake: use CMAKE_SYSTEM_PROCESSOR instead of CMAKE_OSX_ARCHITECTURES for mlx.metallib install (#13696) The CMake condition for installing mlx.metallib checks CMAKE_OSX_ARCHITECTURES, but this variable is only set when explicitly passed - not auto-detected. The arm64 build was missing this flag, causing the metallib to not be installed, which then caused codesign to fail on the unexpanded glob pattern. --- CMakeLists.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/CMakeLists.txt b/CMakeLists.txt index 7724faa03..b17630918 100644 --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -199,7 +199,7 @@ if(MLX_ENGINE) # Install the Metal library for macOS arm64 (must be colocated with the binary) # Metal backend is only built for arm64, not x86_64 - if(APPLE AND CMAKE_OSX_ARCHITECTURES MATCHES "arm64") + if(APPLE AND CMAKE_SYSTEM_PROCESSOR STREQUAL "arm64") install(FILES ${CMAKE_BINARY_DIR}/_deps/mlx-build/mlx/backend/metal/kernels/mlx.metallib DESTINATION ${OLLAMA_INSTALL_DIR} COMPONENT MLX)