build: Tweak linker flags for zstd

Signed-off-by: Hiroshi Hatake <[email protected]>

Author: cosmo0920

CMakeLists.txt

@@ -579,14 +579,38 @@ add_subdirectory(${FLB_PATH_LIB_MINIZ} EXCLUDE_FROM_ALL)
 
 # Zstd (zstd)
 if(FLB_PREFER_SYSTEM_LIB_ZSTD)
-  find_package(PkgConfig)
-  pkg_check_modules(LIBZSTD libzstd>=1.4.8)
+  find_package(ZSTD 1.4.8 QUIET)
+  if(ZSTD_FOUND)
+    set(LIBZSTD_FOUND TRUE)
+    set(LIBZSTD_LIBRARIES ZSTD::ZSTD)
+  else()
+    find_package(PkgConfig)
+    pkg_check_modules(LIBZSTD libzstd>=1.4.8)
+    if(LIBZSTD_FOUND)
+      include_directories(${LIBZSTD_INCLUDE_DIRS})
+      link_directories(${LIBZSTD_LIBRARY_DIRS})
+      add_library(ZSTD::ZSTD INTERFACE IMPORTED)
+      set_property(TARGET ZSTD::ZSTD PROPERTY
+        INTERFACE_LINK_LIBRARIES "${LIBZSTD_LIBRARIES}")
+      set(LIBZSTD_LIBRARIES ZSTD::ZSTD)
+    endif()
+  endif()
 endif()
-if(LIBZSTD_FOUND)
-  include_directories(${LIBZSTD_INCLUDE_DIRS})
-  link_directories(${LIBZSTD_LIBRARY_DIRS})
-else()
+
+if(NOT LIBZSTD_FOUND)
   include(cmake/zstd.cmake)
+  target_compile_definitions(libzstd_static PRIVATE ZSTD_NAMESPACE=flbzstd)
+
+  if(MSVC)
+    # MSVC: no ELF visibility attributes; exports are opt-in via dllexport/DEF.
+    # Nothing special to add here.
+  else()
+    target_compile_definitions(libzstd_static PRIVATE ZSTD_HIDE_SYMBOLS=1)
+    target_compile_options(libzstd_static PRIVATE -fvisibility=hidden)
+    set(FLB_EXCLUDE_LIBS_ALL_NEEDED TRUE)
+  endif()
+
+  set(LIBZSTD_LIBRARIES libzstd_static)
 endif()
 
 # ring buffer library