Merge pull request #133 from bernardladenthin/claude/static-linking-msvcp140-TCpZ1

Use static MSVC runtime (/MT) for Windows builds

Author: bernardladenthin

CMakeLists.txt

@@ -1,7 +1,17 @@
-cmake_minimum_required(VERSION 3.14)
+cmake_minimum_required(VERSION 3.15)
 
 project(jllama CXX)
 
+# Use static MSVC runtime (/MT) instead of the default DLL runtime (/MD).
+# This embeds the C++ runtime into jllama.dll so msvcp140.dll / vcruntime140.dll
+# are not required on the end-user's machine.
+# Must be set before any FetchContent_MakeAvailable() so that llama.cpp and all
+# other subprojects inherit the same CRT choice (mixing /MT and /MD in a single
+# link is a linker error).
+if(MSVC)
+    set(CMAKE_MSVC_RUNTIME_LIBRARY "MultiThreaded$<$<CONFIG:Debug>:Debug>" CACHE STRING "" FORCE)
+endif()
+
 include(FetchContent)
 
 set(BUILD_SHARED_LIBS ON)
@@ -273,8 +283,9 @@ if(BUILD_TESTING)
         GIT_REPOSITORY https://github.com/google/googletest.git
         GIT_TAG        v1.15.2
     )
-    # Prevent GTest from overriding our compiler/linker settings on Windows
-    set(gtest_force_shared_crt ON CACHE BOOL "" FORCE)
+    # Keep GTest on the same CRT as the rest of the project.
+    # OFF means GTest respects CMAKE_MSVC_RUNTIME_LIBRARY (static /MT here).
+    set(gtest_force_shared_crt OFF CACHE BOOL "" FORCE)
     FetchContent_MakeAvailable(googletest)
 
     enable_testing()