StarMirror/xlnt
1
0
Fork 0
mirror of https://github.com/tfussell/xlnt.git synced 2024-03-22 13:11:17 +08:00
Code Issues Projects Releases Wiki Activity

define _CRT_SECURE_NO_WARNINGS for libstudxml to supress MSVC C4996

Browse Source
This commit is contained in:
Thomas Fussell 2017-03-20 19:24:34 -04:00
parent f862de1ff7
commit 6b119bec0c
1 changed files with 15 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
third-party/libstudxml/CMakeLists.txt vendored
View File

@ -28,17 +28,29 @@ set(GENX
find_package(EXPAT REQUIRED)
add_library(libstudxml STATIC ${LIBSTUDXML} ${GENX})
target_compile_definitions(libstudxml
PUBLIC LIBSTUDXML_STATIC_LIB=1
PUBLIC LIBSTUDXML_EXTERNAL_EXPAT=1)
target_include_directories(libstudxml
PUBLIC ${LIBSTUDXML_ROOT_DIR}
PUBLIC ${EXPAT_INCLUDE_DIRS})
target_link_libraries(libstudxml
PRIVATE ${EXPAT_LIBRARIES})
target_link_libraries(libstudxml PRIVATE ${EXPAT_LIBRARIES})
# Prevent warning C4996 caused by strcpy, strncpy, sprintf in genx
# TODO: would it be better to define this only in genx.c?
if(MSVC)
target_compile_definition(libstudxml PRIVATE _CRT_SECURE_NO_WARNINGS=1)
endif()
# Build with -fPIC when xlnt is a shared library
# TODO: is this still necessary? try removing
if(NOT STATIC)
set_target_properties(libstudxml PROPERTIES POSITION_INDEPENDENT_CODE 1)
else()
endif()
# When xlnt is a static library, assume expat will be linked statically too
# TODO: is this a valid assumption?
if(STATIC)
target_compile_definitions(libstudxml PUBLIC XML_STATIC=1)
endif()