Доработал сборку

Author: orefkov

CMakeLists.txt

@@ -73,8 +73,22 @@ add_simstr()
 target_link_libraries(simjson_simjson PUBLIC simstr::simstr)
 
 if(SIMJSON_IN_SHARED)
+    # Всем объявляем, что мы будем в shared библиотеке
     add_compile_definitions(SIMJSON_IN_SHARED)
+    # Себе объявим, что мы должны экспортировать функции
     target_compile_definitions(simjson_simjson PRIVATE SIMJSON_EXPORT)
+
+    # Пример построения shared библиотеки, которая включит в себя simjson и simstr.
+    # Включается опцией SIMJSON_IN_SHARED. Как при этом создать экзешник, пользующийся этой dll - пример в tests/CMakeLists.txt]]
+    # Нельзя объявить библиотеку без исходников, добавим просто .h файл
+    add_library(simjson_dll SHARED include/simjson/json.h)
+    # simstr нельзя просто так добавить как whole_archive, CMake при сборке simjson уже пометил её как DEFAULT
+    target_link_libraries(simjson_dll PRIVATE
+        "$<LINK_LIBRARY:WHOLE_ARCHIVE,simjson::simjson>"
+        simstr_simstr
+    )
+    # Но можно для нужного таргета переопределить её как whole_archive
+    set_property(TARGET simjson_dll PROPERTY LINK_LIBRARY_OVERRIDE "WHOLE_ARCHIVE,simstr_simstr")
 endif(SIMJSON_IN_SHARED)
 
 if(SIMJSON_BUILD_TESTS)
@@ -93,17 +107,6 @@ if(SIMJSON_BUILD_TESTS)
     add_subdirectory(tests)
 endif()
 
-#[[
-# Пример построения shared библиотеки, которая включит в себя simjson и simstr.
-# Необходимо также задать опцию SIMJSON_IN_SHARED. Как при этом создать экзешник,
-# пользующийся этой dll - пример в tests/CMakeLists.txt
-if (SIMJSON_IN_SHARED)
-    add_library(simjson_dll SHARED include/simjson/json.h)
-    target_link_libraries(simjson_dll PRIVATE "$<LINK_LIBRARY:WHOLE_ARCHIVE,simjson::simjson>,simstr_simstr>")
-    set_property(TARGET simjson_dll PROPERTY LINK_LIBRARY_OVERRIDE "WHOLE_ARCHIVE,simstr_simstr")
-endif()
-#]]
-
 # ---- Install rules ----
 
 if(NOT CMAKE_SKIP_INSTALL_RULES)

tests/CMakeLists.txt

@@ -1,21 +1,21 @@
 # CMakeList.txt : CMake project for core_as, include source and define
 # project specific logic here.
 #
+
 add_executable(test_json test_json.cpp)
-target_link_libraries(test_json simjson::simjson GTest::gtest_main)
 
-#[[
-# Пример создания экзешника, если simjson и simstr экспортируются из dll.
-# Так как мы подключаем только simjson_dll, и входящие в неё либы не видны
-# нужно вручную задать каталоги include и стандарт C++20
-if (SIMJSON_IN_SHARED)
+if (NOT SIMJSON_IN_SHARED)
+    target_link_libraries(test_json simjson::simjson GTest::gtest_main)
+else()
+    # Пример создания экзешника, если simjson и simstr экспортируются из dll.
+    # Так как мы подключаем только simjson_dll, и входящие в неё либы не видны
+    # нужно вручную задать каталоги include и стандарт C++20]]
     target_link_libraries(test_json simjson_dll GTest::gtest_main)
     get_target_property(SIMJSON_INCLUDES simjson::simjson INCLUDE_DIRECTORIES)
     get_target_property(SIMSTR_INCLUDES simstr::simstr INCLUDE_DIRECTORIES)
     target_include_directories(test_json PRIVATE ${SIMJSON_INCLUDES} ${SIMSTR_INCLUDES})
     target_compile_features(test_json PUBLIC cxx_std_20)
 endif()
-#]]
 
 add_test(NAME test_json COMMAND test_json WORKING_DIRECTORY ${CMAKE_BINARY_DIR})