wip cleanup zstd_vendor package

Signed-off-by: Karsten Knese <[email protected]>

rosbag2_compression/CMakeLists.txt

@@ -34,13 +34,12 @@ add_library(${PROJECT_NAME}_zstd
 target_include_directories(${PROJECT_NAME}_zstd
   PUBLIC
   $<BUILD_INTERFACE:${CMAKE_CURRENT_SOURCE_DIR}/include>
-  $<INSTALL_INTERFACE:include>
-  ${zstd_vendor_INCLUDE_DIRS})
-target_link_libraries(${PROJECT_NAME}_zstd ${zstd_vendor_LIBRARIES})
+  $<INSTALL_INTERFACE:include>)
 ament_target_dependencies(${PROJECT_NAME}_zstd
   rcpputils
   rcutils
-  rosbag2_storage)
+  rosbag2_storage
+  zstd_vendor)
 # Causes the visibility macros to use dllexport rather than dllimport.
 # This is appropriate when building the dll but not consuming it.
 target_compile_definitions(${PROJECT_NAME}_zstd

zstd_vendor/CMakeLists.txt

@@ -28,7 +28,7 @@ macro(build_zstd)
 
   include(ExternalProject)
   # The CMakeLists.txt file for zstd is in a subdirectory.
-  # We need to configure the CMake command to build from there instead. 
+  # We need to configure the CMake command to build from there instead.
   ExternalProject_Add(zstd-1.4.4
     URL https://github.com/facebook/zstd/archive/v1.4.4.zip
     URL_MD5 3a5c3a535280b7f4dfdbd739fcc7173f
@@ -43,13 +43,13 @@ macro(build_zstd)
     DESTINATION .)
 endmacro()
 
-if (NOT zstd_FOUND OR "${zstd_VERSION}" VERSION_LESS 1.4.4)
+if(NOT zstd_FOUND OR "${zstd_VERSION}" VERSION_LESS 1.4.4)
   build_zstd()
 else()
   message(STATUS "Found Zstd, skipping build.")
 endif()
 
 ament_export_libraries(zstd)
-ament_export_include_directories(include)
+#ament_export_include_directories(include)
 
-ament_package()
+ament_package(CONFIG_EXTRAS zstd_vendor-extras.cmake)

zstd_vendor/zstd_vendor-extras.cmake

@@ -0,0 +1 @@
+list(APPEND zstd_vendor_TARGETS zstd::zstd)