Skip to content

adds missing zstd and lz4 deps that mcap lib needs#59

Closed
sinaaghli wants to merge 2 commits intomainfrom
pr-fix-missing-dep
Closed

adds missing zstd and lz4 deps that mcap lib needs#59
sinaaghli wants to merge 2 commits intomainfrom
pr-fix-missing-dep

Conversation

@sinaaghli
Copy link

in a client repo that uses data_tamer as a thirdparty submodule the CI fails with

CMake Error at /usr/share/cmake-3.22/Modules/FindPackageHandleStandardArgs.cmake:230 (message):
Could NOT find ZSTD (missing: ZSTD_LIBRARY ZSTD_INCLUDE_DIR)
Call Stack (most recent call first):
/usr/share/cmake-3.22/Modules/FindPackageHandleStandardArgs.cmake:594 (_FPHSA_FAILURE_MESSAGE)
dependencies/data_tamer/data_tamer_cpp/3rdparty/mcap/cmake/FindZSTD.cmake:32 (FIND_PACKAGE_HANDLE_STANDARD_ARGS)
dependencies/data_tamer/data_tamer_cpp/3rdparty/mcap/CMakeLists.txt:4 (find_package)


Failed <<< f_libs [36.6s, exited with code 1]
Aborted <<< f_msgs [36.6s]

this pr tries to resolve the cmake's error in not finding ZSTD!

@sinaaghli sinaaghli closed this Aug 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant