]> git.djapps.eu Git - pkg/ggml/sources/llama.cpp/commitdiff
tests : add _CRT_SECURE_NO_WARNINGS for WIN32 (#8231)
authorDaniel Bevenius <redacted>
Thu, 4 Jul 2024 10:53:42 +0000 (12:53 +0200)
committerGitHub <redacted>
Thu, 4 Jul 2024 10:53:42 +0000 (13:53 +0300)
This commit adds the compile definition `_CRT_SECURE_NO_WARNINGS`
to the root cmake subproject.

The motivation for this is that currently the following warnings are
displayed when compiling the tests and common cmake subprojects:
```console
test-llama-grammar.cpp
C:\llama.cpp\src\.\llama.cpp(1406,77): warning C4996: 'strerror':
This function or variable may be unsafe. Consider using strerror_s
instead. To disable deprecation, use _CRT_SECURE_NO_WARNINGS. See
online help for details.
[C:\llama.cpp\build\tests\test-llama-grammar.vcxproj]
...
```

This compile definition is currently set for the `src` subproject
and this change moves into the root cmake project so that it is applied
to all cmake subprojects.

CMakeLists.txt
src/CMakeLists.txt

index e3a0cc369e364e6f414d780e9f4d146052d9549b..d95414d710f56691a594fc9a3faee8431bc49579 100644 (file)
@@ -42,6 +42,10 @@ endif()
 
 option(BUILD_SHARED_LIBS "build shared libraries" ${BUILD_SHARED_LIBS_DEFAULT})
 
+if (WIN32)
+    add_compile_definitions(_CRT_SECURE_NO_WARNINGS)
+endif()
+
 #
 # option list
 #
index ccb607e56d336b74a726b36746e494cbb69ce0b1..c2049df79c212b1d07a9cddd3d68d9269449c6ec 100644 (file)
@@ -1,7 +1,5 @@
 # TODO: should not use this
 if (WIN32)
-    add_compile_definitions(_CRT_SECURE_NO_WARNINGS)
-
     if (BUILD_SHARED_LIBS)
         set(CMAKE_WINDOWS_EXPORT_ALL_SYMBOLS ON)
     endif()