fix FlatBuffers's shared lib target name

and refactor

Author: ta7mid

CMakeLists.txt

@@ -280,7 +280,7 @@ if (REFLECTCPP_BSON OR REFLECTCPP_CHECK_HEADERS)
 
     find_package(bson 1.25.1 CONFIG REQUIRED NAMES bson bson-1.0)
 
-    # The package and target names have changed in libbson v2
+    # The package and target names have changed since v2
     # https://github.com/mongodb/mongo-c-driver/releases/tag/2.0.0
     if(TARGET bson::${linkage_type})
         target_link_libraries(reflectcpp PUBLIC bson::${linkage_type})
@@ -316,22 +316,29 @@ if (REFLECTCPP_CBOR OR REFLECTCPP_CHECK_HEADERS)
     include_directories(PUBLIC ${jsoncons_INCLUDE_DIRS})
 endif ()
 
+set(arrow_lib Arrow::arrow_${linkage_type})
 if (REFLECTCPP_CSV OR REFLECTCPP_CHECK_HEADERS)
-    if (NOT TARGET Arrow::arrow_${linkage_type})
+    if (NOT TARGET "${arrow_lib}")
         find_package(Arrow CONFIG REQUIRED)
     endif()
 
-    target_link_libraries(reflectcpp PUBLIC Arrow::arrow_${linkage_type})
+    target_link_libraries(reflectcpp PUBLIC "${arrow_lib}")
 endif ()
 
 if (REFLECTCPP_FLEXBUFFERS OR REFLECTCPP_CHECK_HEADERS)
     list(APPEND REFLECT_CPP_SOURCES
         src/reflectcpp_flexbuf.cpp
     )
-    if (NOT TARGET flatbuffers::flatbuffers)
+
+    set(flatbuf_lib flatbuffers::flatbuffers)
+    if(REFLECTCPP_BUILD_SHARED)
+        string(APPEND flatbuf_lib "_shared")
+    endif()
+    if (NOT TARGET "${flatbuf_lib}")
         find_package(flatbuffers CONFIG REQUIRED)
     endif ()
-    target_link_libraries(reflectcpp PUBLIC flatbuffers::flatbuffers)
+
+    target_link_libraries(reflectcpp PUBLIC "${flatbuf_lib}")
 endif ()
 
 if (REFLECTCPP_MSGPACK OR REFLECTCPP_CHECK_HEADERS)
@@ -348,18 +355,19 @@ if (REFLECTCPP_MSGPACK OR REFLECTCPP_CHECK_HEADERS)
     endif()
 endif ()
 
+set(parquet_lib Parquet::parquet_${linkage_type})
 if (REFLECTCPP_PARQUET OR REFLECTCPP_CHECK_HEADERS)
-    if (NOT TARGET Arrow::arrow_${linkage_type})
+    if (NOT TARGET "${arrow_lib}")
         find_package(Arrow CONFIG REQUIRED)
     endif()
 
-    if (NOT TARGET Parquet::parquet_${linkage_type})
+    if (NOT TARGET "${parquet_lib}")
         find_package(Parquet CONFIG REQUIRED
             NAMES Parquet Arrow  # Conan merges the Parquet package into Arrow
         )
     endif()
 
-    target_link_libraries(reflectcpp PUBLIC Arrow::arrow_${linkage_type} Parquet::parquet_${linkage_type})
+    target_link_libraries(reflectcpp PUBLIC "${arrow_lib}" "${parquet_lib}")
 endif ()
 
 if (REFLECTCPP_TOML OR REFLECTCPP_CHECK_HEADERS)