Skip to content

Commit 9d0b137

Browse files
committed
tests: Don't build memory unit tests on Windows
It looks like on Windows, unit tests can only access methods exposed via the LLAMA_API export. The test-memory.cpp tests are accessing internal APIs intentionally, so don't build on Windows. Branch: HybridCache Signed-off-by: Gabe Goodhart <ghart@us.ibm.com>
1 parent 60ed266 commit 9d0b137

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

tests/CMakeLists.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -108,6 +108,7 @@ if (NOT WIN32)
108108
llama_build_and_test(test-grammar-integration.cpp)
109109
llama_build_and_test(test-llama-grammar.cpp)
110110
llama_build_and_test(test-chat.cpp)
111+
llama_build_and_test(test-memory.cpp)
111112
# TODO: disabled on loongarch64 because the ggml-ci node lacks Python 3.8
112113
if (NOT ${CMAKE_SYSTEM_PROCESSOR} MATCHES "loongarch64")
113114
llama_build_and_test(test-json-schema-to-grammar.cpp WORKING_DIRECTORY ${CMAKE_CURRENT_SOURCE_DIR}/..)
@@ -145,7 +146,6 @@ endif()
145146
llama_build_and_test(test-log.cpp)
146147
llama_build_and_test(test-chat-template.cpp)
147148
llama_build_and_test(test-regex-partial.cpp)
148-
llama_build_and_test(test-memory.cpp)
149149

150150
# this fails on windows (github hosted runner) due to curl DLL not found (exit code 0xc0000135)
151151
if (NOT WIN32)

0 commit comments

Comments
 (0)