clickhouse-cpp icon indicating copy to clipboard operation
clickhouse-cpp copied to clipboard

[feature request] Conan support

Open itrofimow opened this issue 2 years ago • 2 comments

Even though it's pretty convenient to build because of the great documentation you guys provide, would be awesome if the library could be used as a conan package.

For instance, we are trying to package userver framework into conan, and as of now clickhouse driver is missing from it, because it uses the clickhouse-cpp library underneath, which is not present in cci

itrofimow avatar Oct 13 '22 01:10 itrofimow

Seems like it's a WIP already https://github.com/ClickHouse/clickhouse-cpp/pull/219

itrofimow avatar Oct 13 '22 01:10 itrofimow

Huston, we have had a problem... This fix for clang https://github.com/ClickHouse/clickhouse-cpp/blob/master/CMakeLists.txt#L43-L64 works correctly when you build clickhouse-cpp from cmakelists. But if we install the package, we'll drop this fix. So we cannot compile executable depends on clickhouse-cpp. For example, conan test_package cannot be compiled while lib compiles ok https://pastebin.com/raw/4s64dqTm What should be better, manage this from conan recipe or move this fix to clang_fix.cmake and include everywhere? Related conan pr https://github.com/conan-io/conan-center-index/pull/17312 P.S. This issue does not relate to conan, we'll have the same with cmake install command cc @Enmk

xakod avatar Apr 30 '23 09:04 xakod

It seems clickhouse-cpp exists on conan. maybe close this issue?

cthulhu-irl avatar Jun 30 '24 21:06 cthulhu-irl

fyi @Enmk, close please

xakod avatar Jul 04 '24 06:07 xakod