actions/itgmania212121
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Make tomcrypt workaround for #107 apply everywhere

Browse Source 
I'm seeing the same warning building with clang on Linux.
This commit is contained in:
Arthur Eubanks
2023-10-23 22:45:06 -07:00
committed by teejusb
parent 28b7659a09
commit 7f76a1f5b9
1 changed files with 5 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files
extern/CMakeProject-tomcrypt.cmake
Vendored
+5 -6
View File
@@ -469,14 +469,13 @@ if(WITH_NO_ROLC_TOMCRYPT)
target_compile_definitions("tomcrypt" PRIVATE LTC_NO_ROLC)
endif()
# Fixes build failures due to clang warnings
# See itgmania/itgmania#107 for details
# TODO(teejusb/natano): Remove these two lines once these issues have been fixed upstream.
set_property(TARGET "tomcrypt" PROPERTY C_STANDARD 90)
set_property(TARGET "tomcrypt" PROPERTY C_STANDARD_REQUIRED ON)
if(APPLE)
set_property(TARGET "tomcrypt" PROPERTY XCODE_ATTRIBUTE_GCC_NO_COMMON_BLOCKS "YES")
# Fixes build failures due to warnings on macOS >= 13.3
# See itgmania/itgmania#107 for details
# TODO(teejusb/natano): Remove these two lines once these issues have been fixed upstream.
set_property(TARGET "tomcrypt" PROPERTY C_STANDARD 90)
set_property(TARGET "tomcrypt" PROPERTY C_STANDARD_REQUIRED ON)
endif()
if(MSVC)
target_compile_definitions("tomcrypt" PRIVATE _CRT_SECURE_NO_WARNINGS)