cpp: CMake build #1443
Unanswered
ethanmusser
asked this question in
General
Replies: 1 comment
-
Note some relevant discussion from @james-rms in #1442 (comment). |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
-
The C++ build
README.md
calls out the third-partyolympus-robotics/mcap_builder
project for building with CMake.mcap/cpp/README.md
Lines 95 to 97 in fcd77ea
olympus-robotics/mcap_builder
is quite a simple CMake wrapper that does the following:FetchContent
lz4
andzstd
depedndencies<mcap/mcap.hpp>
headerThis is some fairly typical CMake boilerplate and is not particularly complicated. Why not support this as a first-class feature in the project itself? CMake is a very common build tool.
Instead of a wrapper library as in
olympus-robotics/mcap_builder
, the CMake build could simply find thelz4
/zstd
dependencies (optional), add anINTERFACE
target, and then install/export that target.Is there some reason that I am not considering for why a CMake build is not provided?
Thanks for reading!
Beta Was this translation helpful? Give feedback.
All reactions